Skip to content

Conversation

@dopplershift
Copy link
Member

Looks like running against numpy 1.24.0rc1 has tweaked the starting point of the barbs by a pixel. Just bump the threshold here.

Description Of Changes

Checklist

Looks like running against numpy 1.24.0rc1 has tweaked the starting
point of the barbs by a pixel. Just bump the threshold here.
@dopplershift dopplershift added Type: Maintenance Updates and clean ups (but not wrong) Area: Tests Affects tests Area: Plots Pertains to producing plots labels Nov 29, 2022
@dopplershift dopplershift added this to the October 2022 milestone Nov 29, 2022
@dopplershift dopplershift requested review from a team and kgoebber as code owners November 29, 2022 00:46
@dopplershift dopplershift requested review from dcamron and removed request for a team November 29, 2022 00:46
@kgoebber
Copy link
Collaborator

Agreed.

@dopplershift
Copy link
Member Author

@kgoebber You can click the green button 😉

@kgoebber
Copy link
Collaborator

It says I am not authorized to merge the PR - otherwise I would!

@dopplershift
Copy link
Member Author

Can you check now?

@kgoebber kgoebber merged commit 50b4ab7 into Unidata:main Nov 29, 2022
@kgoebber
Copy link
Collaborator

All good now.

@dopplershift dopplershift deleted the bump-image branch November 29, 2022 17:35
@dopplershift
Copy link
Member Author

I apparently had a couple things locked down that were not working as I intended. Confusing sometimes because you merging a PR counts as a "push" to main, even though I don't think of it that way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Plots Pertains to producing plots Area: Tests Affects tests Type: Maintenance Updates and clean ups (but not wrong)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nightly build is failing

2 participants