openml / openml-python

Python module to interface with OpenML
https://openml.github.io/openml-python/main/
Other
279 stars 143 forks source link

Added warning to run_model_on_task to avoid duplicates if no authentication #1246

Closed v-parmar closed 1 year ago

v-parmar commented 1 year ago

Added warning to run_model_on_task to avoid duplicates if no authentication

Reference Issue

1210

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 50.00% and project coverage change: -0.08 :warning:

Comparison is base (fb9f9eb) 85.24% compared to head (f7c583b) 85.16%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1246 +/- ## =========================================== - Coverage 85.24% 85.16% -0.08% =========================================== Files 38 38 Lines 5008 5010 +2 =========================================== - Hits 4269 4267 -2 - Misses 739 743 +4 ``` | [Impacted Files](https://codecov.io/gh/openml/openml-python/pull/1246?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openml) | Coverage Δ | | |---|---|---| | [openml/runs/functions.py](https://codecov.io/gh/openml/openml-python/pull/1246?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openml#diff-b3Blbm1sL3J1bnMvZnVuY3Rpb25zLnB5) | `83.85% <50.00%> (-0.41%)` | :arrow_down: | ... and [5 files with indirect coverage changes](https://codecov.io/gh/openml/openml-python/pull/1246/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openml) Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openml). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openml)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

PGijsbers commented 1 year ago

Sorry, missed the update. LGTM. Thanks for your contribution!