Issue #799: UKCM2: Investigate cubes missing coordinate "height2m"#857
Issue #799: UKCM2: Investigate cubes missing coordinate "height2m"#857Ed (mo-gill) wants to merge 5 commits intomainfrom
Conversation
Jared Drayton (mo-jareddrayton)
left a comment
Jared Drayton (mo-jareddrayton)
left a comment
There was a problem hiding this comment.
Do the UKESM1p3 mappings need updating as well? #799 (comment)
I see co2_tavg-h2m-hxy-u has been done but there are some other counterparts still using remove_altitude_coords.
Also, I know the unit testing in processors.py is quite patchy and that the original remove_altitude_coords doesn't have one but it might be worth adding one or a functional test.
|
|
||
| This processor also removes the ``HybridHeightFactory`` from the | ||
| cube, see https://scitools.org.uk/iris/docs/latest/whitepapers/\ | ||
| um_files_loading.html#vertical-coordinates for more details. |
There was a problem hiding this comment.
I might be mangling the URL but I have a feeling this is dead. I'm not familiar with the original whitepaper document but I am assuming this is the closest to an up to date version.
Would be worth updating the remove_altitude_coords docstring with this link as well.
There was a problem hiding this comment.
I would expect to see all the changes to the UKCM2 mappings plugin to be replicated in the UKESM1p3 plugin.
3 participants
Fixes #799