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

feat: ✨ pass down resume to download_folder #275

Closed
wants to merge 1 commit into from

Conversation

melMass
Copy link

@melMass melMass commented Jul 6, 2023

Doesn't change the default behaviour but allow for resumable files inside a folder

@mevinbabuc
Copy link

This should be good to merge right ?

@melMass
Copy link
Author

melMass commented Sep 18, 2023

Indeed I've been using it as the gdown version in a project here: https://github.com/melmass/comfy_mtb

achadwick added a commit to achadwick/gdown that referenced this pull request Dec 14, 2023
Allow folder mode to pass through the resume flag.
This still makes some unnecessary requests, however.

Duplicates wkentaro#275,
but also updates cli.py appropriately.
@achadwick
Copy link
Contributor

This alone doesn't change the behaviour enough to make the feature useful to me, sadly 😞

  • main() in cli.py needs to pass resume=args.continue_ to download_folder when it calls it.
  • I think gdown --continue should additionally skip files that are known to be 100% transferred. This turns out to be good when a folder's download allowance is very limited and/or when connections are likely to be interrupted half-way through

I've tried to address these in #288 in case folks are interested in that.

@melMass
Copy link
Author

melMass commented Dec 15, 2023

I don't use the CLI, just the module but I will close this, I don't think they are interested anyway, and it's a 2 line fix

@melMass melMass closed this Dec 15, 2023
achadwick added a commit to achadwick/gdown that referenced this pull request Feb 17, 2024
Allow folder mode to pass through the resume flag.
This still makes some unnecessary requests, however.

Duplicates wkentaro#275,
but also updates cli.py appropriately.
wkentaro pushed a commit to achadwick/gdown that referenced this pull request May 12, 2024
Allow folder mode to pass through the resume flag.
This still makes some unnecessary requests, however.

Duplicates wkentaro#275,
but also updates cli.py appropriately.
wkentaro pushed a commit that referenced this pull request May 12, 2024
Allow folder mode to pass through the resume flag.
This still makes some unnecessary requests, however.

Duplicates #275,
but also updates cli.py appropriately.
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.

None yet

3 participants