Closed ghost closed 1 year ago
Happened to me too.
https://github.com/nathom/streamrip/blob/2418c4e70f49117b140e52321c059edb78af706f/rip/core.py#L327
Should there really be a continue
?
Or the catch clause for ItemExists
should be moved somewhere deeper in the recursion stack?
@nathom
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
same issue on dev branch as of today,
I did find the issue in case anyone is still interested--
The issue is the album-level call (I think?) to download_item
has no failure handling.
I added a try/catch to swallow the exception with a printout saying "already downloaded album", and that monkey patch combined with the --ignore-db
flag lets you download an artist while skipping over already-downloaded albums.
Problematic line here: https://github.com/nathom/streamrip/blob/42b786b7f3c76caf84d5b461619073335e5863a5/streamrip/media.py#L1998
fix:
try:
self.download... blah blah
except:
secho("skipping over an album", fg="yellow")
hope this helps someone!
is this going to be "fixed" in the next release ? or is it supposed to be the default behavior for some reason ?
is this going to be "fixed" in the next release ? or is it supposed to be the default behavior for some reason ?
same issue on dev branch as of today,
I did find the issue in case anyone is still interested-- The issue is the album-level call (I think?) to
download_item
has no failure handling. I added a try/catch to swallow the exception with a printout saying "already downloaded album", and that monkey patch combined with the--ignore-db
flag lets you download an artist while skipping over already-downloaded albums.Problematic line here:
fix:
try: self.download... blah blah except: secho("skipping over an album", fg="yellow")
hope this helps someone!
How could one implement this? 😑
is this going to be "fixed" in the next release ? or is it supposed to be the default behavior for some reason ?
Have you figured out a solution?
Same issue here...
To implement my fix locally, install the package as usual, locate the install folder (usually under site-packages of your python install), and then find the file called media.py, and add a try-catch on the line mentioned in my previous comment.
Let me know if you have any qs!
Describe the bug
When i try to rip a whole artist, if I stop the download and restart it, it just says "(filename) already exists. Skipping". It doesn't check if other albums are downloaded or not. Not only that, if you're in the middle of downloading an artist, it can just say "something already exists" and you can't properly download the rest.
Command Used
Debug Traceback
Config File
Operating System
Linux
streamrip version
1.9.5
Screenshots and recordings
No response
Additional context
No response