Skip to content

Pin Pandas and NumPy versions#24

Closed
kafitzgerald wants to merge 6 commits intoNCAR:mainfrom
kafitzgerald:ci_testing
Closed

Pin Pandas and NumPy versions#24
kafitzgerald wants to merge 6 commits intoNCAR:mainfrom
kafitzgerald:ci_testing

Conversation

@kafitzgerald
Copy link

In order to get a working environment with uv and CI passing again, we need to pin these two dependencies.

Newer version of NumPy aren't supported by Numba, but install anyway and then break stuff if we don't have the version pin.

And then Pandas 3 and newer will break Bridescaler at the moment (test suite fails) so we should pin that until it's addressed.

@kafitzgerald
Copy link
Author

I don't think I can request reviewers, but tagging @djgagne here.

@kevinyang-cky and I sorted this out this morning.

And here's a CI run on my fork that shows this passing in case that's helpful: https://github.com/kafitzgerald/bridgescaler/actions/runs/21724800653

@kevinyang-cky
Copy link
Collaborator

Thank you @kafitzgerald for helping out with the testing. We have decided to adapt the edits here into #23 and close this one here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants