Skip to content
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

Pint update to stop warning #747

Merged
merged 5 commits into from
Nov 7, 2023
Merged

Pint update to stop warning #747

merged 5 commits into from
Nov 7, 2023

Conversation

kenkehoe
Copy link
Contributor

@kenkehoe kenkehoe commented Nov 3, 2023

  • [X ] Documentation reflects changes
  • [ X] PEP8 Standards or use of linter
  • [ X] Xarray Dataset or DataArray variable naming follows 'ds' or 'da' naming

The Pint library is warning of redefining units. This update correctly adds the desired units and stops redefining units. Also commented out some return statements in plotting testing to stop warnings of future errors.

@zssherman zssherman closed this Nov 3, 2023
@zssherman zssherman reopened this Nov 3, 2023
@zssherman
Copy link
Collaborator

Closing and reopening to run tests as github was down

@AdamTheisen
Copy link
Collaborator

Looks good once the final test passes

@AdamTheisen AdamTheisen merged commit bcb8f62 into ARM-DOE:main Nov 7, 2023
17 checks passed
@kenkehoe
Copy link
Contributor Author

kenkehoe commented Nov 7, 2023

@AdamTheisen what are the chances of getting a release with this update? We are getting a warning and my attempts to stop sending alerts to us about the warning are not working. If we make this update I can stop those warnings sending us emails.

@zssherman
Copy link
Collaborator

@kenkehoe I can get one out today!

@zssherman
Copy link
Collaborator

zssherman commented Nov 7, 2023

@kenkehoe The github and pypi release is out, just waiting on conda-forge to pick up the release

@kenkehoe kenkehoe deleted the pint branch December 11, 2023 21:21
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.

3 participants