search

joshspeagle / dynesty

Dynamic Nested Sampling package for computing Bayesian posteriors and evidences
https://dynesty.readthedocs.io/
MIT License
347 stars 76 forks source link

tqdm testing TODO #255

Closed segasai closed 3 years ago

segasai commented 3 years ago

Checking the coverage of the tests, I noticed that tqdm progress bar code is not covered, because tqdm is not included in requirements.txt. One option is to install tqdm for the tests, but then the non-tqdm code won't be covered. So a solution probably is to do some tricks with module loading/unloading/deleting it from sys.modules to ensure that both code-paths are tested by test_printing.

joshspeagle commented 3 years ago

Should now be resolved as of #316