Lighting in vicinity cell_method alignment for operations#2297
Merged
bayliffe merged 3 commits intometoppv:1.17.xfrom Feb 19, 2026
Merged
Lighting in vicinity cell_method alignment for operations#2297bayliffe merged 3 commits intometoppv:1.17.xfrom
bayliffe merged 3 commits intometoppv:1.17.xfrom
Conversation
…he other model cell methods and they don't all get thrown away.
Contributor
Author
|
Note these test failures are also present in a null change PR pointing at 1.17.x and appear to be unrelated to this work. |
gavinevans
approved these changes
Feb 16, 2026
Contributor
gavinevans
left a comment
There was a problem hiding this comment.
Thanks @bayliffe 👍
These changes look fine to me.
2 tasks
MoseleyS
approved these changes
Feb 17, 2026
Member
MoseleyS
left a comment
There was a problem hiding this comment.
Tests run locally - all fine.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In the global data lightning diagnostic that is created within improver we manually create the cell method.
The method is stated as "Sum" with a capital s, which doesn't match the lower case "sum" in the UK model diagnostics with which is blended. When the cubes are combined before merging the data the cell_methods are checked in the MergeCubes plugin and they all get thrown away.
This PR:
Testing:
Testing in the suites
Metadata in model suites
The tables below demonstrate that the metadata is now consistent across the various model suites by the time we get to the latestlevel2 feeding steps (blendcycles). The 3-hour diagnostic differs at the combine step as it feeds again from the standardise step before then having the cell method modified to the expected once again.
lightning1h (T+12 as an example)
lightningvic1h (T+12 as an example)
lightningvic3h (T+12 as an example)
Metadata in blending suite
lightningvic1h (T+12 as an example)
lightningvic3h (T+12 as an example)
lightningvic3h (T+122 as an example beyond the UK model contributions)