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

number format in Update progress.py #269

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Javierete
Copy link

Updated line 275 to allow for the number format to be introduced in the "monitor" and "monitor_end" parameters.

In the current code, monitor = ("{count:,}/{total:,} [{percent:3.1%}] - processing "), throws an error.

By adding these mods, the error is gone and the display show all the python formats available. e.g. below:

|████████████████████████████████████████| 1,000/1,000 [100.00%] - ### COMPLETED ### in 15.8s (63.41/s)

I hope it doesn't ruins something else in the code.

Updated line 275 to allow for the number format to be introduced in the "monitor" and "monitor_end" parameters.

In the current code, monitor = ("{count:,}/{total:,} [{percent:3.1%}] - processing "), throws an error. 

By adding these mods, the error is gone and the display show all the python formats available. e.g. below: 

|████████████████████████████████████████| 1,000/1,000 [100.00%] - ### COMPLETED ###  in 15.8s (63.41/s)

I hope it doesn't ruins something else in the code.
@Javierete
Copy link
Author

as per comment above :) Great job btw. love the library mate! 👍

@Javierete
Copy link
Author

you could also replace with int() instead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant