Show progress during check-setup
#359
Comments
|
Yeah, |
Conditions on the Nextstrain CLI version being requested, as aarch64 builds¹ will only be available going forward for new releases. Assumes the next release will be 8.2.0! Disables Rosetta check when using an aarch64 build, as the checks will be done at the runtime-level instead by `nextstrain setup` and `nextstrain check-setup`.² We must wait to merge this until after the first release (i.e. 8.2.0) of the changes it depends on above. ¹ <#357> ² <#359>
|
I didn't realize there was an existing issue for this. I hacked together d4e5a78 a while back and this is what it looks like, before and after. Note that only conda and ambient are fully unbuffered, but the same can apply to other runtimes as well. demo.movIf others think this is a good direction, I can turn it into a proper PR. |
|
@victorlin Generally in favor of that direction. Note that I've made some changes to the setup/runner tests in #407 (896c2a6), but they're straightforward renames and I can rebase on top of your changes for this if needed. Probably I should pull out several of the ancillary changes from that branch into a new one and get them merged before the workflows-as-programs features. |
nextstrain check-setuptakes almost a minute despite having a fast M1 Pro. There's no progress indicator which might be confusing to users who are running it for the first time.It might be good to give an indication how long this check takes. E.g. "Testing your setup... (this takes about a minute)"
Even better would be some sort of progress indication after each runtime is checked, maybe outputting it or indicating that it has been checked successfully.
The text was updated successfully, but these errors were encountered: