-
Notifications
You must be signed in to change notification settings - Fork 35
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
Fix CI #366
Fix CI #366
Conversation
As predicted in #345 (comment) : it looks like the signs in |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Unpinning dask seems to work perfectly fine (tested this locally). Which means that this #242 is probably no longer relevant. |
@@ -36,14 +36,14 @@ dependencies: | |||
- scikit-image | |||
- seaborn>=0.12.1 | |||
# parallelization | |||
- distributed==2021.1.0 | |||
- dask==2021.1.0 |
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.
keeping these pinned would prevent us from increasing the python version
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.
Thanks @IAlibay, LGTM. And sorry I did not have time to look into this before.
Thanks @RMeli ! |
UserGuide CI has been broken for ~2 months.. let's try to fix it
In this PR:
📚 Documentation preview 📚: https://mdanalysisuserguide--366.org.readthedocs.build/en/366/