Skip to content

Various improvements to transect capability#1992

Open
James W (jwarner8) wants to merge 10 commits intomainfrom
ml_transect
Open

Various improvements to transect capability#1992
James W (jwarner8) wants to merge 10 commits intomainfrom
ml_transect

Conversation

@jwarner8
Copy link
Contributor

Broad collection of improvements to the transect capability in CSET, addressing the following issues/enhancement requests:

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.

@jwarner8 James W (jwarner8) self-assigned this Mar 26, 2026
@jwarner8 James W (jwarner8) added the enhancement New feature or request label Mar 26, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 26, 2026

Coverage

@jwarner8
Copy link
Contributor Author

James W (jwarner8) commented Mar 26, 2026

Status 26/03
Modifications to transects have now been made
image

Started plumbing the aggregation side of things, need to debug why all models are being loaded for one model aggregation. It seems to load model 1 and 2, i.e. the parbaked recipe looks like

file_paths:

  • ~/cylc-run/cset_workflow/run11/share/cycle/*/data/1
  • ~/cylc-run/cset_workflow/run11/share/cycle/*/data/2

this leads to

Constraint doesn't produce single cube: ConstraintCombination(ConstraintCombination(ConstraintCombination(Constraint(), Constraint(cube_func=<function generate_cell_methods_constraint.<locals>.check_no_aggregation at 0x148eceefaae0>), <built-in function and_>), Constraint(name='x_wind'), <built-in function and_>), Constraint(coord_values={'pressure': <function generate_level_constraint.<locals>.<lambda> at 0x148eceefac40>}), <built-in function and_>)
0: x_wind / (m s-1) (time: 16; pressure: 28; latitude: 186; longitude: 142)
1: x_wind / (m s-1) (time: 16; pressure: 28; latitude: 186; longitude: 142)
2: x_wind / (m s-1) (time: 8; pressure: 13; latitude: 281; longitude: 321)
3: x_wind / (m s-1) (time: 8; pressure: 13; latitude: 281; longitude: 321)

A separate problem here is that cubes don't merge in a given model, but primary issue here is two models being loaded. Need to determine if other aggregation recipes do the same.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant