-
-
Notifications
You must be signed in to change notification settings - Fork 211
add task_type to list_runs #857
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #857 +/- ##
==========================================
Coverage ? 88.38%
==========================================
Files ? 37
Lines ? 4298
Branches ? 0
==========================================
Hits ? 3799
Misses ? 499
Partials ? 0
Continue to review full report at Codecov.
|
mfeurer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, thanks a lot. Could you please add this to the changelog, too?
|
|
||
| 0.10.1 | ||
| ~~~~~~ | ||
| * ADD #857: Adds task type ID to list_runs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually my mistake (as I didn't update this yet), but since we made a new release, you need to put this under a new header.
* Fixing broken links (#864) * Adding license to each source file (#862) * Preliminary addition of license to source files * Adding license to almost every source file * add task_type to list_runs (#857) * add task_type to list_runs * length of run change * changelog * changes in progress rst * Prepare new release (#868)
Reference Issue
What does this PR implement/fix? Explain your changes.
API call to list runs https://www.openml.org/api/v1/run/list/limit/10 - returns task type, while
python API does not.
added task type to list_runs call in python API
How should this PR be tested?
runs.list_runs(size=10)
Any other comments?