Skip to content

Pin pandas to 2.3.3 in analysis env#391

Open
mnlevy1981 wants to merge 1 commit intoNCAR:mainfrom
mnlevy1981:update_pins
Open

Pin pandas to 2.3.3 in analysis env#391
mnlevy1981 wants to merge 1 commit intoNCAR:mainfrom
mnlevy1981:update_pins

Conversation

@mnlevy1981
Copy link
Collaborator

Description of changes:

Pandas wasn't explicitly pinned, and recent environments were including 3.0.0 which was causing issues in xr.Dataset.to_netcdf()

Fixes #388

All PRs Checklist:

  • Have you followed the guidelines in our Contributor's Guide?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you made sure that the pre-commit checks passed (#8 in Adding Notebooks Guide)?
  • Have you successfully tested your changes locally when running standalone CUPiD?
  • Have you tested your changes as part of the CESM workflow?
  • Once you are ready to have your PR reviewed, have you changed it from a Draft PR to an Open PR?

Pandas wasn't explicitly pinned, and recent environments were including 3.0.0
which was causing issues in xr.Dataset.to_netcdf()
Copy link
Collaborator

@wwieder wwieder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Copy link
Collaborator

@justin-richling justin-richling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me

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.

Need to pin pandas to 2.3.3

3 participants