-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extend UM vertical plotting to model level #697
Extend UM vertical plotting to model level #697
Conversation
ensure to review #638 first |
coverage report failing. This should be solved by addressing the coverage report issue in PR #638 |
ensure to review #638 first |
43bf53f
to
9ad955d
Compare
477c4ca
to
10f69ee
Compare
a87b854
to
d0455d0
Compare
cset-workflow/includes/lfric_deterministic_domain_mean_vertical_profile_series.cylc
Outdated
Show resolved
Hide resolved
cset-workflow/includes/lfric_deterministic_domain_mean_vertical_profile_series.cylc
Outdated
Show resolved
Hide resolved
978687a
to
081a001
Compare
Branch rebased. |
corrected recipe to reflect new vertical level constraint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple minor help text comments, but otherwise happy to approve, and then ready to merge.
src/CSET/recipes/generic_domain_mean_vertical_profile_modellevel_series.yaml
Outdated
Show resolved
Hide resolved
698ab0a
to
118daf4
Compare
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
…el_series.yaml Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Fixes #658
Um based vertical plotting on model levels
UM based spatial contour plots on model levels
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.
docs/source/changelog.rst