iris icon indicating copy to clipboard operation
iris copied to clipboard

Partial collapse of multi-dim string coords: take 2

Open rcomer opened this issue 1 month ago • 2 comments

🚀 Pull Request

Description

This is the same bugfix as #4294. In there, I also tried to add type hints following my interpretation of the dev guide. This turned out to be complicated and I suspect is the main reason the PR has not made progress.

I think it would be good to fix the bug regardless of typing questions, so here is a version without the type hints which I'm hoping will be an easier review.

Fixes #3653.

If you have a 2D string coordinate on a cube, and collapse one of the dimensions spanned by that coordinate, the coordinate's points (and bounds, if any) are now serialised along the collapsed dimension. This seems to have been missed when partial collapse was implemented for the numeric coordinates.


Consult Iris pull request check list


Add any of the below labels to trigger actions on this PR:

  • https://github.com/SciTools/iris/labels/benchmark_this

rcomer avatar May 15 '24 09:05 rcomer