Skip to content
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

Add option for rotating copies of pole polygons when using poly_area #200

Closed
ngs333 opened this issue Dec 7, 2022 · 1 comment · Fixed by #209
Closed

Add option for rotating copies of pole polygons when using poly_area #200

ngs333 opened this issue Dec 7, 2022 · 1 comment · Fixed by #209
Assignees

Comments

@ngs333
Copy link
Contributor

ngs333 commented Dec 7, 2022

This feature is related to work done for resolving issue #44 .
As issue 44 indicates, the function poly_area yields discontinuities of the area calculated for polygons near the pole.
Also as work for that issue indicates, one can improve on the area calculated for the pole polygons by function poly_area
so that area values are no longer discontinuous and in fact are almost the same as what would be calculated using the GCA (great circle area algorithm) (Note that GCA cannot necessarily be applied to all exchange grid cells, thus the existance of poly_area function). This is done by (merely) making a copy of the polar polygon, rotating it away from the pole to some other earth surface location, and calculating the area of the rotated polygon. Although the work for issue 44 is not complete, NCTools apps should have the feature of a user specified option to rotate copies of polar polygons for the purpose of calculating the area via function poly_area.

@ngs333 ngs333 self-assigned this Dec 7, 2022
@ngs333
Copy link
Contributor Author

ngs333 commented Oct 25, 2023

The work for this was done and merge in PR 209.

@ngs333 ngs333 closed this as completed Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant