• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Ouranosinc / xclim / 8743914296 / 2
92%
main: 92%

Build:
DEFAULT BRANCH: main
Ran 18 Apr 2024 08:12PM UTC
Files 70
Run time 2s
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

18 Apr 2024 07:58PM UTC coverage: 90.002% (-0.009%) from 90.011%
8743914296.2

push

github

web-flow
Remove columns that are all NaNs in make_criteria (#1713)

<!--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:
- [x] This PR addresses an already opened issue (for bug fixes /
features)
    - This PR fixes #xyz
- [x] Tests for the changes have been added (for bug fixes / features)
- [x] (If applicable) Documentation has been added / updated (for bug
fixes / features)
- [x] CHANGES.rst has been updated (with summary of main changes)
- [x] Link to issue (:issue:`number`) and pull request (:pull:`number`)
has been added

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

* Methods such as kmeans clustering will crash if there are NaNs in the
data. This is something that should happen if only a subset of
realizations are NaNs, but we should be safe to remove the columns that
are NaN across all realizations.

### Does this PR introduce a breaking change?

* No


### Other information:

8651 of 9612 relevant lines covered (90.0%)

1.8 hits per line

Source Files on job run-{{ matrix.tox-env }}-opt-slow - 8743914296.2
  • Tree
  • List 0
  • Changed 2
  • Source Changed 0
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 8743914296
  • fb666c90 on github
  • Prev Job for on main (#8743487394.2)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc