Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the song name matching rules in YouTube music #1518

Merged
merged 1 commit into from
May 11, 2022

Conversation

opleonnn
Copy link
Contributor

Title

Update the song name matching rules in YouTube music

Description

Song name matching rules are too strict after matching ISRC code in youtube music. For example a song called Setting Fires - BOXINBOX & Lionsize Remix in Spotify, but called Setting Fires (BOXINBOX & Lionsize Remix) in YouTube Music.
I tested 200 songs and this is my playlist:
https://open.spotify.com/playlist/4OJx800ojOiUCqhL8WRfCQ
https://open.spotify.com/playlist/75DbXkdkz166RgqaomP32E
I found this helped me find the correct version of a dozen songs that were in the wrong version,and the other correct version search results don't change

Related Issue

#1517

Motivation and Context

Improves the probability of downloading the correct version of a song

How Has This Been Tested?

I tested 200 songs and this is my playlist:
https://open.spotify.com/playlist/4OJx800ojOiUCqhL8WRfCQ
https://open.spotify.com/playlist/75DbXkdkz166RgqaomP32E

Screenshots (if appropriate)

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have read the CONTRIBUTING document
  • I have read the CORE VALUES document
  • I have added tests to cover my changes
  • All new and existing tests passed

@opleonnn
Copy link
Contributor Author

opleonnn commented May 11, 2022

I don't know why the test failed.
image

@xnetcat xnetcat merged commit 210bf3c into spotDL:master May 11, 2022
@xnetcat
Copy link
Member

xnetcat commented May 11, 2022

I don't know why the test failed. image

Tests are broken, and I don't have time to fix them. Code looks fine

lukaszb pushed a commit to lukaszb/spotify-downloader that referenced this pull request May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants