Skip to content

Remove redundant baseline tests in test_image. #28118

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

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Apr 22, 2024

... in prevision for the "stop renormalizing to 0-1 prior to resampling" patch, which will lead to some baseline changes.

This removes

  • test_image_interps (testing interpolations); subsumed by test_imshow_masked_interpolation (which tests more interpolations and also masking).
  • test_imshow (testing the extent kwarg, despite the unclear name of the test -- see the original commit, 387fb64); subsumed by more complex tests that also check the extent kwarg, e.g. test_image_cliprect and test_rotate_image.

PR summary

PR checklist

... in prevision for the "stop renormalizing to 0-1 prior to resampling"
patch, which will lead to some baseline changes.

This removes
- test_image_interps (testing interpolations); subsumed by
  test_imshow_masked_interpolation (which tests more interpolations and
  also masking).
- test_imshow (testing the extent kwarg, despite the unclear name of the
  test -- see the original commit, 387fb64); subsumed by more complex
  tests that also check the extent kwarg, e.g. test_image_cliprect and
  test_rotate_image.
@tacaswell tacaswell added this to the v3.10.0 milestone Apr 23, 2024
@tacaswell tacaswell merged commit eb62d69 into matplotlib:main Apr 23, 2024
51 checks passed
@anntzer anntzer deleted the ti branch April 23, 2024 06:30
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