Skip to content

Handle latitude/longitude bounds properly when making polygons#49

Merged
mx-moth merged 2 commits intomainfrom
bounds
Jan 11, 2023
Merged

Handle latitude/longitude bounds properly when making polygons#49
mx-moth merged 2 commits intomainfrom
bounds

Conversation

@mx-moth
Copy link
Contributor

@mx-moth mx-moth commented Jan 11, 2023

Previously the bounds were ignored, with polygons being constructed by finding midpoints between face centres. This approach is still used as a fallback, but if a dataset defines bounds for a coordinate, those bounds are used to construct the polygons.

This means CF grid topologies with 2D coordinates can have properly defined edge cells, if the bounds are defined in the dataset.

@mx-moth mx-moth self-assigned this Jan 11, 2023
Previously the bounds were ignored, with polygons being constructed by
finding midpoints between face centres. This approach is still used as
a fallback, but if a dataset defines bounds for a coordinate, those
bounds are used to construct the polygons.

This means CF grid topologies with 2D coordinates can have properly
defined edge cells, if the bounds are defined in the dataset.
@mx-moth mx-moth merged commit 151c3d1 into main Jan 11, 2023
@mx-moth mx-moth deleted the bounds branch January 11, 2023 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant