search

ziotom78 / Healpix.jl

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

Issue with Resolution struct in query functions #112

Closed 3r3m0s closed 5 months ago

3r3m0s commented 5 months ago

Giving the queryDiscRing function implausible values (e.g. in degree) with a radius larger than pi throws the wrong error because the field name for the pixel count of a given resolution hasn't been updated for this case: using Healpix nside = 8 m = HealpixMap{Float64, RingOrder}(nside) res=Resolution(nside) pix=queryDiscRing(res,5, 180, 5)

gives the error: ERROR: LoadError: type Resolution has no field npix

ziotom78 commented 5 months ago

Hi @3r3m0s , thanks for the report! The master branch has a fix, might you please check that this works? If it is so, I'll release version 4.2.2.

3r3m0s commented 5 months ago

Hi @ziotom78 I just checked, it works now! Screenshot 2024-05-07 124905

3r3m0s commented 5 months ago

Sorry maybe you should close it..