Skip to content

Area average time series#1968

Open
mo-tomosevans wants to merge 2 commits intomainfrom
area_averages
Open

Area average time series#1968
mo-tomosevans wants to merge 2 commits intomainfrom
area_averages

Conversation

@mo-tomosevans
Copy link
Copy Markdown

@mo-tomosevans mo-tomosevans commented Mar 13, 2026

Include area average time series plots capability.
Addresses #1910.

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Ensure rose-suite.conf.example has been updated if new diagnostic added.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

@mo-tomosevans mo-tomosevans self-assigned this Mar 13, 2026
@mo-tomosevans mo-tomosevans linked an issue Mar 13, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 13, 2026

Coverage

@mo-tomosevans mo-tomosevans added enhancement New feature or request R2O Trials Items of significance to the Met Office Research to Operations Team. labels Mar 13, 2026
@mo-tomosevans
Copy link
Copy Markdown
Author

Screenshot (17)

The image attached was generated for air temperature, having used the AREA_AVG method. This is for one cycle, not yet introduced a method/recipe for running over multiple data file inputs.

@mo-tomosevans
Copy link
Copy Markdown
Author

Having difficulty introducing existing aggregation methods to create aggregated area average plots.
When file path is pointing to operational UKV data at <path>/*.pp, ensure_aggregatable_cube does not produce an aggregated cube and fails, outputting

ValueError: ("Cube should have 'forecast_period' and 'forecast_reference_time' dimension coordinates."

I tried the pre-existing generic_surface_domain_mean_time_series_case_aggregation_*.yaml recipes with the same data and they all produced the above error, so I think it may be part of a wider issue for how the archived ukv data is loaded in read_cubes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request R2O Trials Items of significance to the Met Office Research to Operations Team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Area average plots

1 participant