Ouranosinc / xsdba / 21648734949 / 6
79%
main: 79%

DEFAULT BRANCH: main
Ran
Files 23
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 76.086% (+0.07%) from 76.013%
21648734949.6

push

github

web-flow 
add permissive option and fix for 0 (#194)

<!-- Please ensure the PR fulfills the following requirements! -->
<!-- If this is your first PR, make sure to add your details to the
AUTHORS.rst! -->
### Pull Request Checklist:
- [ ] This PR addresses an already opened issue (for bug fixes /
features)
  - This PR fixes #xyz
- [ ] (If applicable) Documentation has been added / updated (for bug
fixes / features).
- [ ] (If applicable) Tests have been added.
- [ ] CHANGELOG.rst has been updated (with summary of main changes).
- [ ] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added.

### What kind of change does this PR introduce?

* Fix clip for 0. When we had a 0 in float32 and we clipped to the next
float32, it would become a 0 (float 32) again in `data_prime`. If we
cast it to float64, it could stay not exactly 0.
  * our tests did not catch this because the inputs were in 64.
  * The casting behavior is different for numpy <2.0, so I pin it above.
* I have encountered climate models with hurs above 100... I have found
that the data_request suggested to clip the data, but not everybody did
it.
(https://earthscience.stackexchange.com/questions/22850/why-are-the-cmip6-model-outputs-for-relative-humidity-not-scaled-to-between-1-10)
* I am thinking maybe we can do the clipping ourselves, so I added that
option with "permissive".


### Does this PR introduce a breaking change?
yes, the behavior will be different for 0.

### Other information:

2733 of 3592 relevant lines covered (76.09%)

0.76 hits per line

Source Files on job run-Python3.12-conda - 21648734949.6