-
Notifications
You must be signed in to change notification settings - Fork 4
Playlist downloads fail when tracks are unavailable for streaming #57
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
Comments
Hello @Fonta.
Does the process reach the end ? Thanks for your feedback 🙏 |
Thanks for reporting 👍 I'll open a tiddl issue for this point. |
For the records : oskvr37/tiddl#66 |
Hello @Fonta This issue should be fixed in latest release : 0.1.12. (Thanks to Tiddl author) Let me know if all is fine :) Regards. |
@cstaelen Tested 0.1.12. If I open a bash shall, it's unable to find the tiddl command. tiddl is available within the venv though. But I don't know if this is causing the error. |
Seems weird, I made 2 fresh install, docker pull and stuff, and I've got no issue, no clue 🤔
|
@cstaelen Think I found the issue. Recreated the container and found that the value for the PATH environment variable changed. |
@cstaelen Running into something else. Some track downloads give an error
|
Thanks for the feedback. |
@cstaelen After executing "pip install ffmpeg-python" in "/root/.venvs" the downloading works. |
Ok great ! You're right, about ffmpeg-python, it's a python binding for ffmpeg, both seem to be necessary (apk + pip binary). I'll fix soon in next release 0.1.13. Thank you so much for your help 🙌 |
@Fonta i wait for your feedback before closing this issue :) |
@cstaelen All seems to work now! Awesome job 💯 |
Sorry for opening the ticket again, but I have just tried to download some albums both in high or master quality, and always getting the same error: MASTER
HIGH
The manual install of ffmpeg-python has no effect as I am already on the latest image version. Any idea? Thanks!! |
Hello @rusty1281 Could please give me the url of the content you try to DL please ? Thank you :) |
@cstaelen It's 72 Seasons by Metallica. Don't know why Tidarr is unable to find https://tidal.com/browse/album/288984589 While browsing to the address does work: |
Yes, that was 72S by Metallica but it seems to fail for all their albums for some reason? |
@rusty1281 Did you try the url I gave you? |
I just got back and tested with a few other artists and it all works fine. Not sure what the deal is with Metallica. |
I did, loads up grey thumbnails as well. No matter, it clearly has nothing to do with this issue at least not in this updated version. Tnx for the assist |
Weird 🤔 |
Ok I got the same issue. Does it happen with other MAX album ? |
Seems to be related to #62 (pls continue the discussion in this new issue) Tidarr allow us to download content available in your logged in Tidal player. Both albums (https://tidal.com/browse/track/339078049 and http://www.tidal.com/album/33969843) are accessible on the public Tidal album page, but in my logged in Tidal player, they aren't. Are you able to find those albums in your logged-in Tidal player ? Thanks for your feedback :) |
@cstaelen The second link to the album though: In my opinion this is an Tidal issue. |
My bad, there was a typo in first URL, I wanted to talk about this one : https://tidal.com/browse/album/288984589 |
Yep, that album link gives the exact same error. |
When tracks are unavailable for streaming, the downloading of the playlist fails.
It should either simply skip the unavailable tracks or continue with the rest of the list when tiddl fails to download a single track.
Perhaps this could even be a setting, like fail on x download failures?
Either way it should notify about the tracks being unavailable for download.
Within Tidal the unavailable tracks look like this:
The text was updated successfully, but these errors were encountered: