-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
Relax dependencies #324
Relax dependencies #324
Conversation
In #322, most codex dependencies were upgraded to fairly recent versions. This PR relaxes the dask and numpy requirements somewhat, but numba remains at 0.60, mostly because there's a numpy >= 2 dependency. Pinging @landmanbester @JSKenyon for review here as these upgrades might be aggressive for Quartical and pfb-imaging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good on my side, happy with the upgrades
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QuartiCal has bracketed dependencies so this won't break it. I will do a QuartiCal release at some point to update to this version and iron out any bugs (but the tests already pass).
Thanks for taking a look |
Tests added / passed
If the pre-commit tests fail, install and
run the pre-commit hooks in your development
virtuale environment:
Fully documented, including
HISTORY.rst
for all changesand one of the
docs/*-api.rst
files for new APITo build the docs locally:
pip install -r requirements.readthedocs.txt cd docs READTHEDOCS=True make html
📚 Documentation preview 📚: https://codex-africanus--324.org.readthedocs.build/en/324/