Open LennartPurucker opened 6 months ago
Attention: Patch coverage is 85.13011%
with 240 lines
in your changes missing coverage. Please review.
Project coverage is 83.88%. Comparing base (
d26a911
) to head (de983ac
). Report is 1 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
openml/datasets/dataset.py | 87.20% | 22 Missing :warning: |
openml/_api_calls.py | 75.64% | 19 Missing :warning: |
openml/cli.py | 0.00% | 18 Missing :warning: |
openml/datasets/functions.py | 86.76% | 18 Missing :warning: |
openml/tasks/task.py | 77.63% | 17 Missing :warning: |
openml/study/functions.py | 72.88% | 16 Missing :warning: |
openml/utils.py | 82.22% | 16 Missing :warning: |
openml/testing.py | 75.80% | 15 Missing :warning: |
openml/config.py | 82.71% | 14 Missing :warning: |
openml/extensions/sklearn/extension.py | 91.89% | 12 Missing :warning: |
... and 16 more |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
There are some problems with the history between
main
anddevelop
. We can clean this up (for example, by branching offmain
again), but Lennart proposed we simply abandon the two branch system in favor of using tags for releases. I personally also would prefer this option, as it's less error prone. We could keep themain
branch around for archival reasons. The documentation pages would need an update.