-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Add tqdm #662
Add tqdm #662
Conversation
Nice to see this here! I'd change from what you wrote originally, though:
to:
What do you think? To further expatiate on why this is awesome, Using as a CLI pipe: $ 7z a -bd -r backup.7z docs/ | grep Compressing |
tqdm --total $(find docs/ -type f | wc -l) --unit files >> backup.log
100%|███████████████████████████████▉| 8014/8014 [01:37<00:00, 82.29files/s] or in a script: from tqdm import tqdm
for i in tqdm(range(1000)):
pass In summary,
-- Casper, main tqdm dev |
@casperdcl no, use:
Simplicity is tqdm's #1 feature. |
@vinta this has had more that 20 votes for ages (since last year), but please don't merge this in until we release |
@vinta I don't think it's worth waiting any longer - happy for you to go ahead and merge this in. At a few million downloads per month, 10k+ stars and sourcerank 22 (see below) it's probably silly for me to hold you back. As I'd said in #662 (comment),
I'll open a new PR |
This needs to be rebased. |
this doesn't need to be rebased @techtonik @vinta #1293 supercedes this |
@casperdcl It does. Could you fix the conflicts? |
Why this framework/library/software/resource is awesome?
Add a progress meter to your loops in a second
Closing issue #648 Add tqdm
Vote for this pull request
Who agrees that this change should be merged could add your reactions (e.g. 👍) to this pull request.
New library added