search

jhardenberg / smmregrid

A compact regridder using sparse matrix multiplication
Apache License 2.0
8 stars 0 forks source link

Introduce NaN instead of 1e20 after the tensordot #25

Closed oloapinivad closed 8 months ago

oloapinivad commented 8 months ago

Fields which are not masked and includes np.nan are filled with 1e20 values and then this values are kept into the resulting array with some unexpected result. This introduces a small fix that bring back the NaN after the tensordot.

oloapinivad commented 8 months ago

@jhardenberg if you agree I can merge and issue a new release

oloapinivad commented 8 months ago

@jhardenberg did you disabled the tests?

oloapinivad commented 8 months ago

@jhardenberg I added a test. If you approve I can merge and issue a new release

jhardenberg commented 8 months ago

Looks good, let's merge!