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.

Sign up for GitHub

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

Jump to bottom

DeprecationWarning: Conversion of an array with ndim > 0 to a scalar is deprecated, and will error in future. #1671

Closed
zssherman opened this issue Oct 23, 2024 · 2 comments · Fixed by #1673
Closed

DeprecationWarning: Conversion of an array with ndim > 0 to a scalar is deprecated, and will error in future. #1671

zssherman opened this issue Oct 23, 2024 · 2 comments · Fixed by #1673
Assignees
@zssherman
Labels
High Priority
Milestone
Py-ART Version 2.0

Comments

@zssherman
Copy link
Collaborator

zssherman commented Oct 23, 2024 •
edited
Loading

  • Py-ART version: Current
  • Python version: 3.12
  • Operating System: All

Description

/home/runner/work/pyart/pyart/pyart/map/gates_to_grid.py:118: DeprecationWarning: Conversion of an array with ndim > 0 to a scalar is deprecated, and will error in future. Ensure you extract a single element from your array before performing this operation. (Deprecated NumPy 1.25.)

I think I have a solution for some of these, however would we want a check for when the array has multiple values? I think in rare cases we do for mobile radars and I believe we take a mean in the gridding code.
@zssherman zssherman self-assigned this Oct 23, 2024
@zssherman zssherman added this to the Py-ART Version 2.0 milestone Oct 23, 2024
@mgrover1 mgrover1 mentioned this issue Oct 23, 2024
Merged
@mgrover1
Copy link
Collaborator

Good to close with #1673 ?

@zssherman
Copy link
Collaborator Author

Yep! Closing.

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

@zssherman zssherman

Labels
High Priority
Projects
None yet
Milestone
Py-ART Version 2.0
Development

Successfully merging a pull request may close this issue.

FIX: Updates code to fix scalar warnings. zssherman/pyart
2 participants
@zssherman @mgrover1