Skip to content

Conversation

@pp-mo
Copy link
Member

@pp-mo pp-mo commented May 28, 2025

Separate PR direct to main
cf HGWright#4

(
So, this is basically a replacement for
#6350
which itself was a replacement for the original
#6190
)

@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

❌ Patch coverage is 92.15686% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.83%. Comparing base (7927212) to head (0b370aa).
⚠️ Report is 137 commits behind head on main.

Files with missing lines Patch % Lines
lib/iris/fileformats/cf.py 91.48% 1 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6481      +/-   ##
==========================================
+ Coverage   89.82%   89.83%   +0.01%     
==========================================
  Files          90       90              
  Lines       23782    23828      +46     
  Branches     4427     4448      +21     
==========================================
+ Hits        21361    21407      +46     
+ Misses       1672     1670       -2     
- Partials      749      751       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pp-mo pp-mo force-pushed the pp_derived_bounds branch from b2c79b5 to ee74499 Compare May 29, 2025 14:48
@pp-mo pp-mo force-pushed the pp_derived_bounds branch from 923ce97 to ad29c2b Compare May 30, 2025 11:34
@pp-mo pp-mo marked this pull request as ready for review May 30, 2025 11:36
@pp-mo pp-mo force-pushed the pp_derived_bounds branch from ccda746 to 6aa9c81 Compare May 30, 2025 11:38
@pp-mo pp-mo force-pushed the pp_derived_bounds branch from bba4326 to 957161f Compare May 30, 2025 15:09
Copy link
Contributor

@HGWright HGWright left a comment

Choose a reason for hiding this comment

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

Thanks Patrick, it was a big collaborative effort. This is a big part of #3678.

@HGWright HGWright merged commit 92e1e96 into SciTools:main Jun 2, 2025
21 checks passed
@bjlittle bjlittle changed the title Pp derived bounds PP derived bounds Jun 4, 2025
@pp-mo pp-mo deleted the pp_derived_bounds branch June 12, 2025 16:43
@scitools-ci scitools-ci bot removed this from 🚴 Peloton Jul 1, 2025
@HGWright HGWright mentioned this pull request Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants