Add tqdm progress bar option for CLI #854
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
Also this is my first open source contribution! If I did anything wrong please let me know I would love to help out with this project 🕊️ One question I have for the maintainers is whether this should stop all prints within analyze_file or maybe keep errors printed as they are less common / more significant than just "Analyzing x file" or "Completed x file" |
esaltzm
changed the title
Member
|
Thanks for contributing! LGTM |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR improves the user experience when running file analysis by:
Wrapping the analyze_file multiprocessing pool with a tqdm progress bar when show_progress is True, providing real-time feedback on processing progress.
Suppressing standard output from worker processes when show_progress is enabled, resulting in a cleaner progress display.
No changes were made to the analysis logic or results.
Motivation
I decided to open this PR after running the analyzer on a directory with ~50,000 audio files and found the CLI output overwhelming and hard to follow. These changes make it easier to monitor progress and keep the output informative and manageable for large-scale analyses.