Skip to content

TST: ignore shape-mutation deprecation warnings from numpy 2.5 on yt 4.4.x - #5518

Merged
neutrinoceros merged 1 commit into
yt-project:yt-4.4.xfrom
neutrinoceros:4.4.x/warning-filter
Sep 1, 2026
Merged

neutrinoceros merged 1 commit into
yt-project:yt-4.4.xfrom
neutrinoceros:4.4.x/warning-filter

Conversation

@neutrinoceros

Copy link
Copy Markdown
Member

PR Summary

For reference, these warnings are already correctly handled on the dev branch (#5309), but the patch isn't strictly backwards compatible so we chose not to backport it. As a result, ignoring the warning is necessary to fix CI on the backport branch.

PR Checklist

  • New features are documented, with docstrings and narrative docs
  • Adds a test for any bugs fixed. Adds tests for new features.

@neutrinoceros neutrinoceros added this to the 4.4.3 milestone Sep 1, 2026
@neutrinoceros neutrinoceros added blocker Highest priority level tests: running tests Issues with the test setup dependencies Pull requests that update a dependency file bug labels Sep 1, 2026
@neutrinoceros

Copy link
Copy Markdown
Member Author

remaining failures are unrelated:

  • 6 tests failing because of another deprecation warning from numpy 2.5 through unyt (I need to handle this one upstream)
  • all image tests failing, presumably because of a change in matplotlib 3.11 we haven't taken into account on the main branch yet

@neutrinoceros
neutrinoceros marked this pull request as ready for review September 1, 2026 09:49

@chrishavlin chrishavlin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

re: the failing image tests, I agree they're unrelated, good to merge thisone.

presumably because of a change in matplotlib 3.11 we haven't taken into account on the main branch yet

of the 10 or so that I looked at, all the failed image diffs show a slight shift of all the axis labels, e.g. the diff for yt.visualization.tests.test_image_comp_2D_plots.TestPhasePlot.test_phaseplot_xax2-yax2-zax2-1-fractional_accumulation :

result-failed-diff

didn't notice anything obvious in the 3.11.1 or 3.11.0 release notes that might be the culprit -- there were a bunch of font/text related updates, but they all looked like new features to me.

@neutrinoceros

Copy link
Copy Markdown
Member Author

I've seen the same change affect astropy's image tests a couple months back. Here are the references
xref: astropy/astropy#19562
xref: matplotlib/matplotlib#30161

@neutrinoceros
neutrinoceros merged commit ea63063 into yt-project:yt-4.4.x Sep 1, 2026
4 of 11 checks passed
@neutrinoceros
neutrinoceros deleted the 4.4.x/warning-filter branch September 1, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocker Highest priority level bug dependencies Pull requests that update a dependency file tests: running tests Issues with the test setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants