-
Notifications
You must be signed in to change notification settings - Fork 283
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
Update CI environment lockfiles #4194
Conversation
I'm not sure if this explains the current nc-time-axis errors, but I know for certain that the versions of nc-time-axis and cftime proposed here are incompatible. Although not recorded in the nc-time-axis requirements, |
@trexfeathers Cool, I'll issue a patch for |
5076f4c
to
0372446
Compare
I can't work out why this is still resolving with nc-time-axis |
EDIT: resolution definitely is picking up an older build - |
@trexfeathers I think this is now fixed, see conda-forge/conda-forge-repodata-patches-feedstock#144, which retro-patches the appropriate It's pretty cool... I didn't know that was possible, thanks @ocefpaf ❤️ |
0372446
to
5078358
Compare
So when we unpin |
We were very "afraid" of repo patches due to possible reproducibility issues. In fact I was very against it. However, they are so useful that we got addicted to them and they are here to stay now 😄 |
FWIW, I'm pretty sure it does: SciTools/nc-time-axis#71 (comment) |
Lockfiles updated to the latest resolvable environment.