Skip to content

Conversation

@janvanrijn
Copy link
Member

@janvanrijn janvanrijn commented Feb 25, 2019

Reference Issue

Fixes #632

What does this PR implement/fix? Explain your changes.

How should this PR be tested?

Any other comments?

@janvanrijn janvanrijn changed the base branch from master to develop February 25, 2019 14:28
@codecov-io
Copy link

codecov-io commented Feb 25, 2019

Codecov Report

Merging #633 into develop will increase coverage by 0.16%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #633      +/-   ##
===========================================
+ Coverage    89.86%   90.03%   +0.16%     
===========================================
  Files           32       32              
  Lines         3208     3232      +24     
===========================================
+ Hits          2883     2910      +27     
+ Misses         325      322       -3
Impacted Files Coverage Δ
openml/study/__init__.py 100% <ø> (ø) ⬆️
openml/study/functions.py 96.87% <100%> (+1.04%) ⬆️
openml/_api_calls.py 88.31% <0%> (+3.89%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0980673...4f60c25. Read the comment docs.

@janvanrijn janvanrijn requested a review from mfeurer February 25, 2019 16:51
size : int, optional
The maximum number of studies to show.
main_entity_type : str, optional
Can be `task` or `run`. In case of `task`, only benchmark suites are
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For rest you should format as:

``'task'`` or ``'run'``

@janvanrijn janvanrijn merged commit 98a73b3 into develop Feb 26, 2019
@janvanrijn janvanrijn deleted the add_#632 branch February 26, 2019 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: study list

4 participants