sdss / mangadap

The MaNGA Data Analysis Pipeline
BSD 3-Clause "New" or "Revised" License
16 stars 9 forks source link

Sampling #43

Closed francbelf closed 6 years ago

francbelf commented 6 years ago

I finally had time to mess around with the new sampling function. It is great! However I think I tracked down a bug at the very end of the _resample_step function (line 584 should be replaced with 591-596 below). Please confirm whether this makes sense to you. Note that the previous version did not correctly deal with the 2D case (the output had the wrong shape).