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

Add argument to explicitly enable progress bar output #310

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

GeorgeNikitinNV
Copy link
Contributor

Problem: after v0.3.0 progress bar is not displayed if dorado is run not from tty.

Solution: added additional argument to explicitly enable progress bar output, not displayed if omitted:

Optional arguments:
...
  -p, --enable-progress-bar     Enable progress bar output to stderr 
...

@GeorgeNikitinNV
Copy link
Contributor Author

GeorgeNikitinNV commented Aug 17, 2023

Not sure why the tests failed, on my machine everything is fine:

❯ ctest --test-dir cmake-build                        
Internal ctest changing into directory: /Users/george.nikitin/dev/dorado/cmake-build
Test project /Users/george.nikitin/dev/dorado/cmake-build
    Start 1: dorado_tests
1/1 Test #1: dorado_tests .....................   Passed  108.46 sec

100% tests passed, 0 tests failed out of 1

Total Test time (real) = 108.52 sec

Also have no clue how to force checks to restart - should I make another commit for that?

@GeorgeNikitinNV
Copy link
Contributor Author

Anyone?

@tijyojwad
Copy link
Collaborator

looks like the checks passed now? I think there's some intermittency in the GitHub builds based on how much disk space the test job gets

@GeorgeNikitinNV
Copy link
Contributor Author

yes, they do

@iiSeymour
Copy link
Member

@GeorgeNikitinNV the diff has your CMakeCache.txt included plus some submodules deltas.. might be easier to resubmit a new request from the head of master.

@palakpsheth
Copy link

would LOVE to see this PR get merged before the next release!

@GeorgeNikitinNV

@Shians
Copy link

Shians commented Feb 11, 2024

This would be very useful for me, I run Dorado on a HPC environment and it'd be very useful to know how it's progressing.

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.

5 participants