igmhub / picca

set of tools for continuum fitting, correlation function calculation, cosmological fits...
GNU General Public License v3.0
29 stars 22 forks source link

P1d nan fix #1038

Closed corentinravoux closed 1 year ago

corentinravoux commented 1 year ago

Fix for the previous PR. In certain cases (no chunks in a specific wavenumber and redshift bin), the new changes crashed. Improvement of the masking of np.nan by the way.

corentinravoux commented 1 year ago

Is it possible to push another minor/fix version for this fix ?

corentinravoux commented 1 year ago

Yes I think the np.nan orginate from empty SNR bins, and the k, z empty bins were already dealt with a warning not accounted for by the previous PR.