search

mweastwood / LibHealpix.jl

A Julia wrapper of the Healpix library.
http://mweastwood.info/LibHealpix.jl/stable/
Other
11 stars 7 forks source link

Julia port of interpolation has some bugs #42

Closed mweastwood closed 6 years ago

mweastwood commented 6 years ago

I'm not entirely sure how my tests didn't catch these, but in my first non-trivial use of this code I found:

https://github.com/mweastwood/LibHealpix.jl/blob/687f8b6ffe3245c00d96dd96b21e4fb9186a76dc/src/rings.jl#L60 should be

        startpix = 2northring*(northring - 1)

and

https://github.com/mweastwood/LibHealpix.jl/blob/687f8b6ffe3245c00d96dd96b21e4fb9186a76dc/src/map.jl#L374 should be 

    θ′, startpix+i1+1, startpix+i2+1, weight

need to sit down and write some more tests apparently