Lighting in vicinity cell_method alignment for master#2296
Merged
bayliffe merged 10 commits intometoppv:masterfrom Feb 26, 2026
Merged
Lighting in vicinity cell_method alignment for master#2296bayliffe merged 10 commits intometoppv:masterfrom
bayliffe merged 10 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
MoseleyS
approved these changes
Feb 26, 2026
gavinevans
added a commit
to gavinevans/improver
that referenced
this pull request
Mar 5, 2026
…sformation * upstream/master: EPPT3121 Add clip option to fine fuel moisture content calculation (metoppv#2317) Edit CLI docstring. (metoppv#2318) Update WeightAndBlend.process() to use *cubes and as_cubelist (metoppv#2314) Lighting in vicinity cell_method alignment for master (metoppv#2296) Add quantile mapping and associated tests (metoppv#2264) MOBT-1069: Using adjacent validity times in SAMOS (metoppv#2310) Mobt991 ApplySAMOS plugin distribution fix (metoppv#2311) Add attribute to realization clustering to record source realizations (metoppv#2305) MOBT-1070: Time-zone safe handling of time point extraction in SAMOS calibration (metoppv#2306) Set START_DATE_DICT to outside largest lag period to silence warnings (metoppv#2304) Eppt2408 implement initialisation process for fire severity iterative parameters (metoppv#2302) FireWeatherIndexBase plugin - change Error to Warning for input values outside expected ranges (metoppv#2301)
gavinevans
added a commit
to gavinevans/improver
that referenced
this pull request
Mar 6, 2026
…ion_update * upstream/master: EPPT3121 Add clip option to fine fuel moisture content calculation (metoppv#2317) Edit CLI docstring. (metoppv#2318) Update WeightAndBlend.process() to use *cubes and as_cubelist (metoppv#2314) Lighting in vicinity cell_method alignment for master (metoppv#2296) Add quantile mapping and associated tests (metoppv#2264)
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)