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.

Sign up for GitHub

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

fix qcvv progress bars when using cli #1072

Merged

richrines1 merged 1 commit into main from qcvv-tqdm-cli

Sep 24, 2024

Contributor

richrines1 commented Sep 23, 2024

currently some progress bars only work correctly in a notebook environment. this is fixed by importing from tqdm.auto instead of tqdm.notebook

also moves patch_tqdm to qcvv/conftest.py to avoid repetition in each test file


fix qcvv progress bars when using cli

2eb8422

cdbf1 approved these changes

View reviewed changes

Contributor

cdbf1 left a comment

I didn't know tqdm.auto was a thing - this is great thanks!

LGTM

richrines1 merged commit fef8d60 into main

20 checks passed

richrines1 deleted the qcvv-tqdm-cli branch

September 24, 2024 13:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment