-
Notifications
You must be signed in to change notification settings - Fork 279
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
Update lock files with associated fixes #5953
Conversation
⏱️ Performance Benchmark Report: f003172Performance shifts
Full benchmark results
Generated by GHA run |
⏱️ Performance Benchmark Report: 6897131Performance shifts
Full benchmark results
Generated by GHA run |
⏱️ Performance Benchmark Report: 14e8bd3Performance shifts
Full benchmark results
Generated by GHA run |
⏱️ Performance Benchmark Report: 893ecd8Performance shifts
Full benchmark results
Generated by GHA run |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5953 +/- ##
==========================================
- Coverage 89.78% 89.78% -0.01%
==========================================
Files 93 93
Lines 23010 23017 +7
Branches 5018 5020 +2
==========================================
+ Hits 20660 20666 +6
Misses 1620 1620
- Partials 730 731 +1 ☔ View full report in Codecov by Sentry. |
⏱️ Performance Benchmark Report: 07c305cPerformance shifts
Full benchmark results
Generated by GHA run |
⏱️ Performance Benchmark Report: 9d4f1b5Performance shifts
Full benchmark results
Generated by GHA run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trexfeathers Nice one 👍
LGTM
Could you also make an issue to remind us to remove the _GeoAxesPatched
behaviour at the next cartopy
release 👍
🚀 Pull Request
Description
Closes #5918
Consult Iris pull request check list
Add any of the below labels to trigger actions on this PR: