manodeep / Corrfunc

⚡️⚡️⚡️Blazing fast correlation functions on the CPU.
https://corrfunc.readthedocs.io
MIT License
163 stars 50 forks source link

Debugging doctest failure #261

Closed lgarrison closed 2 years ago

lgarrison commented 2 years ago

We're getting ERROR: Cannot uninstall 'certifi'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall. e.g. here: https://github.com/manodeep/Corrfunc/pull/258/checks?check_run_id=3886102439

Not sure why, maybe an environment issue? Opening this PR to debug.

lgarrison commented 2 years ago

The issue appears to be that an old (2016) version of certifi is installed with Python 3.6. I guess it's a pinned dependency, because a conda update doesn't update the version. I think it should be fine if we use Python 3.8 for doctests instead, which comes preloaded with a compatible certifi version.

manodeep commented 2 years ago

Should be fine for doctests to run on a more recent version of python. Should we add an entry to the changelog?