search

ziotom78 / Healpix.jl

Healpix library written in Julia
GNU General Public License v2.0
51 stars 18 forks source link

bug in `pix2ringpos` for southern cap #108

Closed marius311 closed 1 year ago

marius311 commented 1 year ago

With v4.2.0, e.g. final pixel should not be first ring (entire southern cap has an issue):

julia> pix2ringpos(Resolution(4), 12*4^2)
(:southcap, 1, 4)
ziotom78 commented 1 year ago

Hi @marius311 , thanks for the report! I implemented that function in 2019, and it seems that I completely skipped the implementation of tests, what a shame!

I fixed the bug and added proper tests, I'll release a new bugfix version very soon.