Introduce multi-variable map plotting#1828
Conversation
than create new subroutines
|
Changeset updated to On the time-averaging aspects, will setup new issue/PR to cover these elements, as think might have benefits/implications across other supported recipes for general capability rather than only specific to this recipe. Ready for review. |
James Warner (jwarner8)
left a comment
There was a problem hiding this comment.
Looks good, few minor suggestions. Highly useful functionality that will need broadening in future (e.g. transects, or aggregation), but advantageous to get this on trunk now.
Co-authored-by: James W <62252918+jwarner8@users.noreply.github.com>
|
Thanks for review comments. See responses above. a) Adopted proposed approach for slice_over_maybe, and seems to preserve required behaviour. |
|
Ahead of merge to main, I ran a ensemble workflow as struggled testing with ensembles on command-line As per discussion above, proposal is to merge this PR to main, with unit testing covering ensemble use-case. Previous testing in development of this change illustrated ensemble multi-variable outputs working correctly. Can review again following improvement to ensemble read reliability. Will merge this PR as-is. |




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