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

Fix progress bars w/ recursive uploads and downloads #473

Conversation

joereuss12
Copy link
Contributor

The progress bars now appear below the debug log as they are running and are not duplicated. Also, disabled progress bars with specified debug log output. Figured you don't need the bars to be printed to a specific log file.

@joereuss12 joereuss12 linked an issue Dec 7, 2023 that may be closed by this pull request
Copy link
Member

@jhiemstrawisc jhiemstrawisc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, and I've never seen such awesome progress bars for all my recursive downloads 😉

Even though I think we're safe to merge while the NPM build step is failing, I'm still going to wait for the tests to pass before I merge.

@haoming29
Copy link
Contributor

@joereuss12 and @jhiemstrawisc Just so you know, Cannon's PR to fix the npm build issue is merged in and we should be good to go

@haoming29 haoming29 added this to the v7.3.0 milestone Dec 8, 2023
The progress bars now appear below the debug log as they are running,
also disabled progress bars with specified debug log output. Figured you
don't need the bars to be printed to a specific log file.
@joereuss12 joereuss12 force-pushed the fix-mpb-recursive-upload-and-download-branch branch from f6f0663 to f4c9407 Compare December 8, 2023 19:45
@joereuss12
Copy link
Contributor Author

Sounds good, pushed again after rebase, hopefully no more issues

@joereuss12 joereuss12 merged commit bfa0e1d into PelicanPlatform:main Dec 8, 2023
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.

Fix progress bars with recursive downloads and uploads
3 participants