cartopy icon indicating copy to clipboard operation
cartopy copied to clipboard

ENH: Add HEALPix and rHEALPix projections

Open greglucas opened this issue 1 year ago • 4 comments
trafficstars

Rationale

This adds the HEALPix and rHEALPix projections from PROJ to Cartopy. https://proj.org/en/9.4/operations/projections/healpix.html https://proj.org/en/9.4/operations/projections/rhealpix.html

This replaces #1008 and allows for flexible globes/radii as well.

closes #882 closes #1008 closes #2133

greglucas avatar Oct 23 '24 17:10 greglucas

For class names, do we want Healpix() or HEALPix()? Similarly for the rectangular version, do we want Rectangular spelled out, or is RHEALPix as I've put it ok?

greglucas avatar Oct 23 '24 17:10 greglucas

Also closes #2133 I think.

rcomer avatar Oct 23 '24 21:10 rcomer

Always enjoy seeing new projections be added! For the name, I would vote for HEALPix() and rHEALPix().

lgolston avatar Oct 26 '24 03:10 lgolston

Always enjoy seeing new projections be added! For the name, I would vote for HEALPix() and rHEALPix().

I think I prefer the capitalization of "R" to indicate it is a class in Python by common convention, even if PROJ does lowercase it. Any other votes/preferences on naming?

I think this is ready for review again, and I added in a review comment about the doc build process when someone doesn't have the latest PROJ installed.

greglucas avatar May 23 '25 03:05 greglucas

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Oct 06 '25 07:10 CLAassistant