search

NIFTy-PPL / NIFTy

Probabilistic programming framework for signal inference algorithms that operate regardless of the underlying grids and their resolutions
https://ift.pages.mpcdf.de/nifty/index.html
GNU General Public License v3.0
13 stars 2 forks source link

Some modules not installed when running tests #13

Closed apizzuto closed 2 months ago

apizzuto commented 2 months ago

ModuleImport errors when running tests as suggested in the README.

========================================== short test summary info ===========================================
ERROR test/test_mpi/test_kl.py
ERROR test/test_mpi/test_sample_list.py
ERROR test/test_mpi/test_sync.py
ERROR test/test_numpy_accuracy.py
ERROR test/test_optimize_kl_out.py
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 5 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
================================== 2 skipped, 4 warnings, 5 errors in 7.93s ==================================

Two tests fail from h5py or ducc0, which are easily installed via pip. The others fail because of mpi4py which requires an MPI installation.

Part of https://github.com/openjournals/joss-reviews/issues/6593#issuecomment-2045195836

Edenhofer commented 2 months ago

Thanks for the report! All optional dependencies are assumed to be installed for testing although nifty8[test] is usually sufficient for developing new tests. Unfortunately, this assumption was not stated in the README. #18 should resolve this.