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

Regenerate all tick test images #1882

Merged
merged 1 commit into from
Sep 22, 2021
Merged

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Sep 21, 2021

The test tolerance is too high, and random other changes have crept in so that some show numbers that are completely different from the reference image.

  • 726ebac changed the Mercator projection in test_set_xyticks to use the default globe, which changed the text of the ticks from degree-ish to metres.
  • Ticks are quite clearly requested at 30° increments in test_set_yticks_cylindrical, so they definitely should not be at ±31/75°.
  • Also, fix some small inconsistencies in the tests.

The test tolerance is too high, and random other changes have crept in
so that some show numbers that are completely different from the
reference image.

* 726ebac changed the `Mercator`
  projection in `test_set_xyticks` to use the default globe, which
  changed the text of the ticks from degree-ish to metres.
* Ticks are quite clearly requested at 30° increments in
  `test_set_yticks_cylindrical`, so they definitely should not be at
  ±31/75°.
* Also, fix some small inconsistencies in the tests.
@QuLogic QuLogic added this to the 0.20.1 milestone Sep 21, 2021
@greglucas greglucas merged commit 019f013 into SciTools:master Sep 22, 2021
@QuLogic QuLogic deleted the fix-tick-tests branch September 22, 2021 19:17
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.

2 participants