-
Notifications
You must be signed in to change notification settings - Fork 68
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
base: master
Are you sure you want to change the base?
Add argument to explicitly enable progress bar output #310
Conversation
Not sure why the tests failed, on my machine everything is fine:
Also have no clue how to force checks to restart - should I make another commit for that? |
Anyone? |
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 |
yes, they do |
@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. |
would LOVE to see this PR get merged before the next release! |
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. |
One intermittent workaround could be to print a bit more verbose output ( |
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: