Lighting in vicinity cell_method alignment for master#2296
Open
bayliffe wants to merge 9 commits intometoppv:masterfrom
Open
Lighting in vicinity cell_method alignment for master#2296bayliffe wants to merge 9 commits intometoppv:masterfrom
bayliffe wants to merge 9 commits intometoppv:masterfrom
Conversation
…he other model cell methods and they don't all get thrown away.
…xclude the addition of a cell method. Note that this change will conflict with master as this code has been extensively modified by the Bureau.
gavinevans
previously approved these changes
Feb 16, 2026
Contributor
gavinevans
left a comment
There was a problem hiding this comment.
Thanks @bayliffe 👍
This looks fine. Acceptance tests pass once this PR is merged with #2297, and these two improver_test_data PRs are merged together: metoppv/improver_test_data#121, metoppv/improver_test_data#122.
MoseleyS
previously approved these changes
Feb 17, 2026
gavinevans
approved these changes
Feb 20, 2026
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.
Additionally, following the generalisation of the vicinity method to allow for mins, means etc. as well as maxes, an additional cell method is added to specify the form of the vicinity that has been applied. When vicinities are applied via the threshold step these cell methods are not added. ENGLUK lightning in vicinity data uses the vicinity CLI and thus gets this extra "area" cell method that is not present on the data from UKVX and ENUKX. This is a further source of difference between the two.
This PR:
Associated suite PR: https://github.com/MetOffice/improver_suite/pull/2623
Associated acceptance test data: metoppv/improver_test_data#121
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)