-
Notifications
You must be signed in to change notification settings - Fork 45
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
[Feature] - nice progress bar during downloading of SLC data #56
Comments
@glshort we've added a One thing to be aware of, is that if you do want to use the Jupyter support, you'll need to be careful with the import b/c you can be running in a Jupyter kernel, but not have all the expected Jupyter dependencies, so we do this: |
@jhkennedy using your link i customized the download.py script in the download folder so that, if the processes params is 1 (the customizeation doesn't support pool() method) now you can see the progres of the download. The function to modify is asf_search/download/download.py: def download_url(url: str, path: str, filename: str = None, session: ASFSession = None ) -> None: |
Is there still interest in an implementation of this? if nothing else, for the top-level loop that shows the total number of files to download would be only a one line change, and it's nice that now you can disable the progress bars with |
Is your feature request related to a problem? Please describe.
Downloading SLC data takes a long time. Would be nice to track.
Describe the solution you'd like
Something like this - https://gist.github.com/wy193777/0e2a4932e81afc6aa4c8f7a2984f34e2 - would be happy to make a pull request if the feature would be welcome. Understand if this is out of scope.
Describe alternatives you've considered
I think there are other progress bars, but am not familiar with them.
Additional context
There is a lot of output during downloading e.g.
Rather be able to track progress more clearly.
edit: realizing this feature request might be a bit hasty as there might be use cases which this is not desired and annoying - but hopefully there might be a smart way to integrate such a feature.
The text was updated successfully, but these errors were encountered: