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

Use tqdm package to display estimated time length when uploading/downloading datasets #142

Open
Andrewq11 opened this issue Jul 18, 2024 · 0 comments
Labels
feature Annotates any PR that adds new features; Used in the release process

Comments

@Andrewq11
Copy link
Contributor

Is your feature request related to a problem? Please describe.

We've recently implemented a progress indicator in the client to provide feedback to users while long running upload/download processes are occurring.

While yaspin provides sufficient feedback to the user regarding the fact that something is happening behind the scenes, tqdm can provide the user with estimated upload/download times for their artifacts. This will be particularly beneficial in the case of large artifacts that can take minutes.

Describe the solution you'd like

We should implement the tqdm progress bar during the upload/download of artifacts to R2. We don't need to fully replace yaspin with tqdm, but it would be beneficial to user experience to add it in the aforementioned scenarios.

Describe alternatives you've considered

Seeing as tqdm is a very popular package for displaying upload/download progress and is already a dependency of polaris-lib, other libraries which can achieve a similar functionality were not considered.

@Andrewq11 Andrewq11 added the feature Annotates any PR that adds new features; Used in the release process label Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Annotates any PR that adds new features; Used in the release process
Projects
None yet
Development

No branches or pull requests

1 participant