Skip to content

Backport PR #14671 on branch v5.0.x (Fix numpy 1.25 deprecation warnings) - #14686

Merged
pllim merged 1 commit into
astropy:v5.0.xfrom
larrybradley:auto-backport-of-pr-14671-on-v5.0.x
Apr 27, 2023
Merged

pllim merged 1 commit into
astropy:v5.0.xfrom
larrybradley:auto-backport-of-pr-14671-on-v5.0.x

Conversation

@larrybradley

Copy link
Copy Markdown
Member

Backport PR #14671 on branch v5.0.x (Fix numpy 1.25 deprecation warnings)

@pllim pllim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@pllim pllim added skip-basebranch-check Skip base branch check for direct backports Manual Backport labels Apr 24, 2023
@pllim

pllim commented Apr 24, 2023

Copy link
Copy Markdown
Member

Gonna close/re-open to make bot happy.

@pllim pllim closed this Apr 24, 2023
@pllim pllim reopened this Apr 24, 2023
@saimn

saimn commented Apr 25, 2023

Copy link
Copy Markdown
Contributor

Tests are failing with

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

Why does this happen only in the v5.0.x backport ?

@pllim

pllim commented Apr 25, 2023

Copy link
Copy Markdown
Member

Probably need to patch more because that code was removed in later branches? 😒

@pllim

pllim commented Apr 25, 2023

Copy link
Copy Markdown
Member

Ugh looks painful. Should we just pin maxversion of numpy in v5.0.x and be done with it?

@pllim

pllim commented Apr 27, 2023

Copy link
Copy Markdown
Member

@WilliamJamieson might follow-up with a patch for the remaining failures. If not, we'll consider pinning a max version of numpy in LTS. Meanwhile, let's just get this in. Doesn't hurt to have either way. Thanks!

@pllim
pllim merged commit 0b5c04b into astropy:v5.0.x Apr 27, 2023
@larrybradley
larrybradley deleted the auto-backport-of-pr-14671-on-v5.0.x branch April 27, 2023 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants