spotDL / spotify-downloader

Download your Spotify playlists and songs along with album art and metadata (from YouTube if a match is found).
https://spotdl.readthedocs.io/en/latest/
MIT License
17.36k stars 1.6k forks source link

Song Download Mismatch (Wrong Download/No Downloaded) #1273

Closed RowanSkie closed 3 years ago

RowanSkie commented 3 years ago

Environment

Version: 3.5.2
Summary: Download your Spotify playlists and songs along with album art and metadata
Home-page: https://github.com/spotDL/spotify-downloader
Author: spotDL Team
Author-email: spotdladmins@googlegroups.com
License: MIT
Location: c:\pl\python39\lib\site-packages
Requires: pytube, beautifulsoup4, mutagen, tqdm, spotipy, ytmusicapi, rapidfuzz, requests, rich
Required-by:

Expected Behaviour

  1. I'm trying to download The Longest Johns' Oak & Ash & Thorn, and expect that it downloads this video: https://music.youtube.com/watch?v=4GT59EqPlUQ&list=RDAMVM4GT59EqPlUQ.
    • The program will convert the whole hour, which isn't really a good thing to see since it's really just a small song, therefore the system to search the song has a wrong download
  2. I'm trying to download chumachechaya vesna/Чумачечая Весна and it doesn't detect the Youtube Music link: https://music.youtube.com/watch?v=A-PjXUzhFDk&list=RDAMVMA-PjXUzhFDk
    • The program will instead error out, and this is possibly actually due to its foreign characters being recognized. Translating it to English creates a "song not found" error on Spotify, and manually typing it creates a "song not found" on Youtube instead.

Actual Behaviour

For Oak & Ash & Thorn, it downloads this 1 hour video: https://www.youtube.com/watch?v=zIHmPlI4JEs For Chumachechaya Vesna, the song can't be found.

Steps to Reproduce

  1. Run "spotdl https://open.spotify.com/track/38FCXf9CTcH9MJKn3WsJ7x"
  2. Run "spotdl https://open.spotify.com/track/2eaSMmKfigsm96aTUJMoIk"

Traceback

For Oak & Ash & Thorn:

D:\Downloads\Audio\Music\Spotdl>spotdl https://open.spotify.com/track/38FCXf9CTcH9MJKn3WsJ7x
Fetching Song...
Searching for: The Longest Johns - Oak & Ash & Thorn
The Longest Johns - Oak & Ash & Thorn              Converting         ------------------------------------------------------------         90% 0:00:02

For Chumachechaya Vesna:

D:\Downloads\Audio\Music\Spotdl>spotdl https://open.spotify.com/track/2eaSMmKfigsm96aTUJMoIk
Fetching Song...
Searching for: Potap & Nastya - Чумачечая весна
Skipping Чумачечая весна (https://open.spotify.com/track/2eaSMmKfigsm96aTUJMoIk) as no match could be found on youtube
D:\Downloads\Audio\Music\Spotdl>spotdl "Potap & Nastya - chumachechaya vesna"
Searching for song "Potap & Nastya - chumachechaya vesna"...
No song matches found on Spotify

D:\Downloads\Audio\Music\Spotdl>spotdl "Potap & Nastya - Чумачечая весна"
Searching for song "Potap & Nastya - Чумачечая весна"...
Searching for: Potap & Nastya - Чумачечая весна
Searching for: Potap & Nastya - Чумачечая весна - Live
Searching for: Potap & Nastya - Чумачечая весна
Could not match any of the results on YouTube

D:\Downloads\Audio\Music\Spotdl>spotdl "Потап и Настя - Чумачечая Весна
Searching for song "Потап и Настя - Чумачечая Весна"...
Searching for: Potap & Nastya - Чумачечая весна
Searching for: Potap & Nastya - Чумачечая весна - Live
Searching for: Potap & Nastya - Чумачечая весна
Could not match any of the results on YouTube

Any other information:

Related to other previous issues I've given, specifically #970 and #1089.

RowanSkie commented 3 years ago

Additional information:

  1. Tried out The Longest Johns feat SKÁLD - Santiano (https://open.spotify.com/track/22C9LUZT1A0BSe09mYrVup). Got the 2:29 Youtube video, instead of the 2:09 video that contained only the raw music.
  2. Tried out Cash Cash - Surrender (https://open.spotify.com/track/3rwdcyPQ37SSsf1loOpux9). Song wasn't detected, and seeing other details, should've taken this video: https://www.youtube.com/watch?v=xAIoh9rxRi8.
  3. Tried out Lisa Hannigan - Amhrán na Farraige (https://open.spotify.com/track/07paTkxx4R7rmiGjqm84RM). Got a 2:42-long song instead of 2:45/2:46.
  4. Tried out De Vet Du - Pantamera (https://open.spotify.com/track/760xwlNMwa6IZGff1eBhFW). Song wasn't detected, but the song is on Youtube.
  5. Tried out Jeckyll & Hyde - Freefall (https://open.spotify.com/track/17Ph3lPJTrq5t7esgouyFv). Song immediately cut off at 3:20 instead of 3:28.
xnetcat commented 3 years ago

I don't think that this is a bug in spotdl. ytmusicapi sometimes returns results without artists so spotdl can't filter out the wrong results that's why you get No matches found error or wrong songs. I am not sure if we can fix this here

Atakku commented 3 years ago

Maybe you could use the length data to sort results with the least length mismatch

xnetcat commented 3 years ago

Maybe you could use the length data to sort results with the least length mismatch

We are already doing this https://github.com/spotDL/spotify-downloader/blob/master/spotdl/search/provider.py#L237

Atakku commented 3 years ago

Maybe you could use the length data to sort results with the least length mismatch

We are already doing this https://github.com/spotDL/spotify-downloader/blob/master/spotdl/search/provider.py#L237

for some reason downloaded an hour long version for me, let me recheck, maybe im not using latest

Atakku commented 3 years ago

seems weird, i'm on latest, and when downloading this track it looked up this one hour version instead

xnetcat commented 3 years ago

seems weird, i'm on latest, and when downloading this track it looked up this one hour version instead

What is your ytmusicapi version? pip show ytmusicapi to check version

from ytmusicapi import YTMusic

client = YTMusic()

song_results = client.search("clouds", filter="songs")
video_results = client.search("clouds", filter="videos")

results = song_results + video_results

for result in results:
    print(result["duration"])

Can you run this script and paste output here? It should look something like this image

RowanSkie commented 3 years ago

seems weird, i'm on latest, and when downloading this track it looked up this one hour version instead

What is your ytmusicapi version? pip show ytmusicapi to check version

from ytmusicapi import YTMusic

client = YTMusic()

song_results = client.search("clouds", filter="songs")
video_results = client.search("clouds", filter="videos")

results = song_results + video_results

for result in results:
    print(result["duration"])

Can you run this script and paste output here? It should look something like this image

Okay, so I did that while he hasn't responded, and here's what I got:

C:\Users\xterm>pip show ytmusicapi
Name: ytmusicapi
Version: 0.16.0
Summary: Unofficial API for YouTube Music
Home-page: https://github.com/sigma67/ytmusicapi
Author: sigma67
Author-email:
License: MIT
Location: c:\pl\python39\lib\site-packages
Requires: requests
Required-by: spotdl

C:\Users\xterm>D:

D:\>cd workbench

D:\Workbench>py test.py
4:32
3:05
4:04
3:00
3:52
3:11
3:50
3:10
4:17
4:26
5:24
4:38
4:14
3:05
3:13
5:37
4:35
2:40
2:41
9:20
4:14
3:04
4:38
3:14
3:50
7:14
4:39
3:04
4:04
3:50
2:57
10:00:00
2:44
57:03
3:35
3:05
56:59
3:50
4:16
3:05
xnetcat commented 3 years ago

Ok, I've did some research

I think I can fix the issue with searching for the first song, but I am not so sure about the second one. The bugfix will probably help with the other problems that you mentioned

https://github.com/xnetcat/spotify-downloader/tree/song-matching-imprv-working

/cc @MikhailZex

RowanSkie commented 3 years ago

Ok, I've did some research

  • First track gets the wrong results because youtube music api returns way more video type results than song results and the correct one isn't even included in the api response.
  • Second track couldn't be found because on spotify the artist name is "Potap & Nastya", while on the yotube it's "Потап и Настя", they are written in different languages that's why spotdl fails

I think I can fix the issue with searching for the first song, but I am not so sure about the second one. The bugfix will probably help with the other problems that you mentioned

https://github.com/xnetcat/spotify-downloader/tree/song-matching-imprv-working

/cc @MikhailZex

I can ignore the second one, to be honest. It's expected, since it's a Russian song.

xnetcat commented 3 years ago

@RowanSkie looks like I've fixed the issues with song matching. Here's the code https://github.com/xnetcat/spotify-downloader/tree/song-matching-imprv-working go ahead and try it but be aware that it's not ready for production yet, I am still fixing bugs

ghost commented 3 years ago

The current matching system goes for the best possible match, there are no conditions specified to "discard obviously wrong results" i.e. results that are clearly wrong to human beings looking at them but technically score a closer match according to the chose match criteria. A good example of something like this when spotdl downloads a "instrumental cover" of a song instead of the slightly longer original from YouTube.

The below results are from my personal retrofitted spotdl copy, if they are accurate you can use my copy.

RowanSkie commented 3 years ago

The current matching system goes for the best possible match, there are no conditions specified to "discard obviously wrong results" i.e. results that are clearly wrong to human beings looking at them but technically score a closer match according to the chose match criteria. A good example of something like this when spotdl downloads a "instrumental cover" of a song instead of the slightly longer original from YouTube.

The below results are from my personal retrofitted spotdl copy, if they are accurate you can use my copy.

Okay, so how do I download your copy to my python? I'm not too sure how to install that. The songs are match, after all.

ghost commented 3 years ago

pip install https://github.com/MikhailZex/spotify-downloader/archive/refs/heads/Retrofit.zip --force-reinstall

RowanSkie commented 3 years ago

I'm gonna have to try xnetcat's version since it failed when I tried Longest Johns' Santiano, and downloaded the wrong Not Alone by Forgét Mej (Should be this one not this one.)

xnetcat commented 3 years ago

pip install https://github.com/xnetcat/spotify-downloader/archive/refs/heads/song-matching-improvements.zip --force-reinstall

RowanSkie commented 3 years ago

Okay, so, I've tried both versions and used the following songs:

MikhailZex's version

  1. Oak & Ash & Thorn - Success
  2. Chumachechaya Vesna - Success
  3. Not Alone (Forgét Mej) - Fail, took Not Alone by Blank
  4. Not Alone (Stellaris Trailer) - Success
  5. Santiano - Fail, no song detected

For xnetcat's version, those that failed got it correct.

What I really like with Zex's version is the fact that it documents the possible errors but it's not... exactly that accurate. Good for personal use, indeed. xnetcat's version is accurate too, except for Chumachechaya Vesna (a Fail with no song detected).

xnetcat commented 3 years ago

@RowanSkie try again now it should correctly download Chumachechaya Vesna. You will have to reinstall spotdl

ghost commented 3 years ago

What I really like with Zex's version is the fact that it documents the possible errors but it's not... exactly that accurate. Good for personal use, indeed. xnetcat's version is accurate too, except for Chumachechaya Vesna (a Fail with no song detected).

Could you elaborate on the accuracy bit. I'd like to improve it if possible.

RowanSkie commented 3 years ago

What I really like with Zex's version is the fact that it documents the possible errors but it's not... exactly that accurate. Good for personal use, indeed. xnetcat's version is accurate too, except for Chumachechaya Vesna (a Fail with no song detected).

Could you elaborate on the accuracy bit. I'd like to improve it if possible.

I can't really explain it without actually looking on the results. Not sure why, but the fact that two songs failed (A Latin song and a song got the wrong song). It's probably just on the detection side of things, don't worry.

ghost commented 3 years ago

Cool. Could ya send me the Spotify links for the two that failed ?

RowanSkie commented 3 years ago

https://open.spotify.com/track/22C9LUZT1A0BSe09mYrVup (Not Alone)

https://open.spotify.com/track/22C9LUZT1A0BSe09mYrVup (Santiano)

ghost commented 3 years ago

https://open.spotify.com/track/22C9LUZT1A0BSe09mYrVup (Not Alone)

https://open.spotify.com/track/22C9LUZT1A0BSe09mYrVup (Santiano)

both links are the same

RowanSkie commented 3 years ago

Whoops. Here's for Not Alone: https://open.spotify.com/track/4jkRBTpdsAszuD1N87OI7B?si=1c681684f73142dc

ghost commented 3 years ago

Spotify Data

song_name = 'Santiano (feat. SKÁLD)'
album_name = 'Santiano (feat. SKÁLD)'
duration = 129.5 #secs
song_artists = ['The Longest Johns', 'SKÁLD']

Note

The current matching system goes for the best possible match, there are no conditions specified to "discard obviously wrong results" i.e. results that are clearly wrong to human beings looking at them but technically score a closer match according to the chose match criteria. A good example of something like this when spotdl downloads a "instrumental cover" of a song instead of the slightly longer original from YouTube.

As you can see the filtering of "obviously wrong results" filters out most of the returned YTM results. You would get better matches for songs like the 2 that went wrong for you but the total fraction of errors would go up too. Currently the fraction of errors is at ~1% of all results (on a sample set of 600 songs I downloaded to test this). I'd just say bad luck overall.

RESULTS Key

YTM Results

DURATION_MATCH SKIP: {
    "album": null,
    "artists": [
        "The Longest Johns"
    ],
    "duration": 150,
    "link": "https://www.youtube.com/watch?v=oWx7O9bUnMg",
    "name": "Santiano (feat. SK\u00c1LD)"
}
----------

DURATION_MATCH SKIP: {
    "album": null,
    "artists": [
        "FlamSparks"
    ],
    "duration": 154,
    "link": "https://www.youtube.com/watch?v=4FEeFobviqo",
    "name": "SK\u00c1LD ft. The Longest Johns | Santiano (Lyrics & Translation)"
}
----------

DURATION_MATCH SKIP: {
    "album": null,
    "artists": [
        "Lyrics First"
    ],
    "duration": 219,
    "link": "https://www.youtube.com/watch?v=6DdRiNmj0wE",
    "name": "The Longest Johns - Santiana (Lyrics) (Best Version) | Away Santiana!"
}
----------

DURATION_MATCH SKIP: {
    "album": null,
    "artists": [
        "The Longest Johns"
    ],
    "duration": 242,
    "link": "https://www.youtube.com/watch?v=IM6cgSORHt8",
    "name": "Lowlands Low | The Longest Johns with @Sean Dagher"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "SK\u00c1LD"
    ],
    "duration": 131,
    "link": "https://www.youtube.com/watch?v=aOeFdpVfb8I",
    "name": "J\u00f6rmungrund"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "The Longest Johns"
    ],
    "duration": 195,
    "link": "https://www.youtube.com/watch?v=8Fow61Zsn2s",
    "name": "Leave Her Johnny | The Longest Johns | Mass Choir Community Video Project"
}
----------

NEW TOP MATCH: {
    "album": null,
    "artists": [
        "Colm McGuinness Music"
    ],
    "duration": 115,
    "link": "https://www.youtube.com/watch?v=9Qpgh63l2n4",
    "name": "Roll, Boys, Roll! (Assassin's Creed IV Black Flag) Shanty"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Longest Johns Gaming"
    ],
    "duration": 6949,
    "link": "https://www.youtube.com/watch?v=XB-ljbYRqg4",
    "name": "It's folk music time! New banjo! - 11/12/2020 Stream Full VOD"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "The Longest Johns"
    ],
    "duration": 213,
    "link": "https://www.youtube.com/watch?v=Dj_0Zz3eOhw",
    "name": "The Cruel Wars | The Longest Johns"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Karl Sternau"
    ],
    "duration": 98,
    "link": "https://www.youtube.com/watch?v=jPjwCTcqqF0",
    "name": "WELLERMAN [German Version by @Der Michel  + instrumental by @Karl Sternau [+ English Translation]"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Vikingskog"
    ],
    "duration": 83,
    "link": "https://www.youtube.com/watch?v=IErcd74z07Q",
    "name": "Mandal-sjanti (Norwegian shanty)"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "SK\u00c1LD"
    ],
    "duration": 138,
    "link": "https://www.youtube.com/watch?v=V3qYr1GgmWY",
    "name": "S\u00f3larlj\u00f3\u00f0"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Stefan W\u00fcnnemann"
    ],
    "duration": 378,
    "link": "https://www.youtube.com/watch?v=4G2bzxNSpsM",
    "name": "W\u00fcnne & Udo spielen: \u201cBraw burn the bridges\u201c von The Tannahill Weavers"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Alex Hamel"
    ],
    "duration": 129,
    "link": "https://www.youtube.com/watch?v=NdakXBoGFPE",
    "name": "Pique le Baleine - AleX Hamel"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Willard Losinger's Music Channel"
    ],
    "duration": 153,
    "link": "https://www.youtube.com/watch?v=KlGVEKV18wM",
    "name": "Willard Losinger Performs \"Santiano\" (French version) w/ Harmonica & Baritone Ukulele Accompaniment"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Rowan the Bard"
    ],
    "duration": 107,
    "link": "https://www.youtube.com/watch?v=Kc2wst6t95c",
    "name": "Hondo Ohnaka - Star Shanty #2"
}
----------

NAME_MATCH SKIP: {
    "album": null,
    "artists": [
        "Le Jammie Dodgere"
    ],
    "duration": 230,
    "link": "https://www.youtube.com/watch?v=273hw_X1o14",
    "name": "Roll Northumbria but your oil tanker is passing through a storm"
}
----------

ALBUM_MATCH SKIP: {
    "album": "Santiano",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 130,
    "link": "https://www.youtube.com/watch?v=i7fWziFIJZo",
    "name": "Santiano (feat. SK\u00c1LD)"
}
----------

ALBUM_MATCH SKIP: {
    "album": "Between Wind And Water",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 175,
    "link": "https://www.youtube.com/watch?v=laY4TcWEbSA",
    "name": "Santiana"
}
----------

NAME_MATCH SKIP: {
    "album": "Santiano",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 123,
    "link": "https://www.youtube.com/watch?v=GvwqNMvyRRM",
    "name": "Tri Martolod"
}
----------

NAME_MATCH SKIP: {
    "album": "Between Wind And Water",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 197,
    "link": "https://www.youtube.com/watch?v=qLOJk3eV1zg",
    "name": "General Taylor"
}
----------

NAME_MATCH SKIP: {
    "album": "Wellerman",
    "artists": [
        "The Longest Johns",
        "Strings Of The London Symphony Orchestra"
    ],
    "duration": 166,
    "link": "https://www.youtube.com/watch?v=au7ifkiSnP4",
    "name": "Wellerman (with Strings Of The London Symphony Orchestra)"
}
----------

NAME_MATCH SKIP: {
    "album": "Between Wind And Water",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 158,
    "link": "https://www.youtube.com/watch?v=uSGpI9ybK9c",
    "name": "Sugar In The Hold"
}
----------

NAME_MATCH SKIP: {
    "album": "Between Wind And Water",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 177,
    "link": "https://www.youtube.com/watch?v=ZObwftsdOIw",
    "name": "Haul Away Joe"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 249,
    "link": "https://www.youtube.com/watch?v=4GT59EqPlUQ",
    "name": "Oak & Ash & Thorn"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 251,
    "link": "https://www.youtube.com/watch?v=uZOJZsxECNw",
    "name": "Here's a Health to the Company"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 245,
    "link": "https://www.youtube.com/watch?v=V57-k-RaAcQ",
    "name": "Bonny Ship the Diamond"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 138,
    "link": "https://www.youtube.com/watch?v=flqS1AB3_vI",
    "name": "Four Hours"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 201,
    "link": "https://www.youtube.com/watch?v=CxdgnEPDDxQ",
    "name": "Ashes"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 164,
    "link": "https://www.youtube.com/watch?v=u3mbWqRXEJ8",
    "name": "Moby Duck"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 172,
    "link": "https://www.youtube.com/watch?v=dKB9loUK-BM",
    "name": "Got No Beard"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 149,
    "link": "https://www.youtube.com/watch?v=0TtxwqQFxEM",
    "name": "Fire & Flame"
}
----------

ALBUM_MATCH SKIP: {
    "album": "Santiana",
    "artists": [
        "Gustavo Steiner"
    ],
    "duration": 112,
    "link": "https://www.youtube.com/watch?v=Rh0LYF7bTEw",
    "name": "Santiana"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 280,
    "link": "https://www.youtube.com/watch?v=cfDp8BjR1w4",
    "name": "The Banks of the Lee"
}
----------

NAME_MATCH SKIP: {
    "album": "Loch Lomond",
    "artists": [
        "The Hound + The Fox"
    ],
    "duration": 206,
    "link": "https://www.youtube.com/watch?v=QiaXrA98ow0",
    "name": "Loch Lomond"
}
----------

NAME_MATCH SKIP: {
    "album": "A Colm Night At Sea",
    "artists": [
        "Colm R. McGuinness"
    ],
    "duration": 103,
    "link": "https://www.youtube.com/watch?v=r5Qkv4wwtxs",
    "name": "Or\u00f3 S\u00e9 do Bheatha 'Bhaile"
}
----------

NAME_MATCH SKIP: {
    "album": "Cures What Ails Ya",
    "artists": [
        "The Longest Johns"
    ],
    "duration": 170,
    "link": "https://www.youtube.com/watch?v=Lw3XY6cDEjg",
    "name": "Hoist up the Thing"
}
----------

FINAL MATCH: Santiano (feat. SKÁLD) = https://www.youtube.com/watch?v=9Qpgh63l2n4
RowanSkie commented 3 years ago

Huh, that's just weird. I mean, this is just the raw audio Youtube video: https://www.youtube.com/watch?v=i7fWziFIJZo. But hey, that explains why it didn't get detected that close.

xnetcat commented 3 years ago

What I really like with Zex's version is the fact that it documents the possible errors

If you want you can open new issue, and request this as a feature

ghost commented 3 years ago

Huh, that's just weird. I mean, this is just the raw audio Youtube video: https://www.youtube.com/watch?v=i7fWziFIJZo. But hey, that explains why it didn't get detected that close.

YTM queries don't necessarily return the exact same results as going to music.youtube.com as running as search. So the base results were working with aren't necessarily the same.

It's the same most if the time but not always.

ghost commented 3 years ago

What I really like with Zex's version is the fact that it documents the possible errors

If you want you can open new issue, and request this as a feature

We already tried. I myself self-closed that PR. Fitting in that PR will disrupt almost every other PR created/drafted.

As such is just porting the search system from the redesign to the current version.

ghost commented 3 years ago

Huh, that's just weird. I mean, this is just the raw audio Youtube video: https://www.youtube.com/watch?v=i7fWziFIJZo. But hey, that explains why it didn't get detected that close.

I tried the same:

RowanSkie commented 3 years ago

Oh, well, I guess it's fixed now. Would it be weird if both versions can be merged in a way that doesn't hit the main build itself?

xnetcat commented 3 years ago

Oh, well, I guess it's fixed now. Would it be weird if both versions can be merged in a way that doesn't hit the main build itself?

Not really

We already tried. I myself self-closed that PR. Fitting in that PR will disrupt almost every other PR created/drafted.

As such is just porting the search system from the redesign to the current version.

ndgnuh commented 2 years ago

Sorry for bumping this issue.

I still find spotdl download the wrong version of the same song sometime

spotdl 'billy joel - pianoman' # piano man works

Maybe add an option to specify the exact youtube link?

Silverarmor commented 2 years ago

@ndgnuh No support is provided for search queries, use Spotify links instead - we are a spotify downloader.

You also are able to pick what youtube video to use: image