-
Notifications
You must be signed in to change notification settings - Fork 283
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
Load performance improvement (ignoring UGRID) #6088
Conversation
* main: Mesh nonexperimental extra (SciTools#6077) Mesh nonexperimental (SciTools#6061) # Conflicts: # lib/iris/experimental/ugrid/cf.py
⏱️ Performance Benchmark Report: 609d5d2Performance shifts
Full benchmark results
Generated by GHA run |
Initial benchmark results are interesting. not all straightforward. |
WHOOPS sorry, I did that to the wrong PR |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6088 +/- ##
=======================================
Coverage 89.77% 89.77%
=======================================
Files 88 88
Lines 23011 23026 +15
Branches 5032 5036 +4
=======================================
+ Hits 20657 20672 +15
Misses 1623 1623
Partials 731 731 ☔ View full report in Codecov by Sentry. |
⏱️ Performance Benchmark Report: 621b9b8Performance shifts
Full benchmark results
Generated by GHA run |
⏱️ Performance Benchmark Report: fd30e4dPerformance 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.
@stephenworsley Looks good! 💯
Just a couple of comments to service, then we can bank this 👍
Co-authored-by: Bill Little <bill.little@metoffice.gov.uk>
* main: Adding monthly and yearly arguments to `guess_bounds` (SciTools#6090)
* upstream/main: Load performance improvement (ignoring UGRID) (SciTools#6088) Adding monthly and yearly arguments to `guess_bounds` (SciTools#6090) Mesh nonexperimental extra (SciTools#6077) Mesh nonexperimental (SciTools#6061) Bump scitools/workflows from 2024.07.4 to 2024.07.5 (SciTools#6076) Updated environment lockfiles (SciTools#6068) Enable UGRID loading always; deprecate PARSE_UGRID_ON_LOAD. (SciTools#6054) Bump scitools/workflows from 2024.07.3 to 2024.07.4 (SciTools#6071)
🚀 Pull Request
Attempts to address performance slowdowns highlighted here #6075