Skip to content

Conversation

@dopplershift
Copy link
Member

Description Of Changes

This gets the suite passing on my M2. There are a couple contour tests that need a higher threshold for some reason, but manual inspection confirms tests are behaving as expected.

Need to fix a couple other tests.

This gets the suite passing on my M2. There are a couple contour tests
that need a higher threshold for some reason, but manual inspection
confirms tests are behaving as expected.
@dopplershift dopplershift added Type: Maintenance Updates and clean ups (but not wrong) Area: Tests Affects tests hacktoberfest-accepted PRs accepted for contributions in hacktoberfest labels Oct 31, 2023
@dopplershift dopplershift added this to the October 2023 milestone Oct 31, 2023
@dopplershift dopplershift marked this pull request as ready for review October 31, 2023 22:35
@dopplershift dopplershift requested review from a team and kgoebber as code owners October 31, 2023 22:35
@dopplershift dopplershift requested review from dcamron and removed request for a team October 31, 2023 22:35
kgoebber
kgoebber previously approved these changes Nov 1, 2023
Copy link
Collaborator

@kgoebber kgoebber left a comment

Choose a reason for hiding this comment

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

Looks good.

@dopplershift dopplershift changed the title MNT: Update some image test thresholds Make tests pass on macOS M2 Nov 1, 2023
@dopplershift
Copy link
Member Author

I've got two more non-image tests failing locally.

Apparently this problematic test fails here too.
Copy link
Member

@dcamron dcamron left a comment

Choose a reason for hiding this comment

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

Passes locally on my M2 as well 👍

@dcamron dcamron merged commit 83308c3 into Unidata:main Nov 1, 2023
@dopplershift dopplershift deleted the fix-apple-tests branch November 1, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Tests Affects tests hacktoberfest-accepted PRs accepted for contributions in hacktoberfest Type: Maintenance Updates and clean ups (but not wrong)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants