Skip to content

added example strang, and more filter options - #793

Merged
mfeurer merged 6 commits into
developfrom
add_example_strang
Oct 2, 2019
Merged

mfeurer merged 6 commits into
developfrom
add_example_strang

Conversation

@janvanrijn

Copy link
Copy Markdown
Member

Reference Issue

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

How should this PR be tested?

Any other comments?

@codecov-io

codecov-io commented Oct 1, 2019 •

Copy link
Copy Markdown

Codecov Report

Merging #793 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #793      +/-   ##
===========================================
+ Coverage    87.71%   87.73%   +0.01%     
===========================================
  Files           36       36              
  Lines         4208     4214       +6     
===========================================
+ Hits          3691     3697       +6     
  Misses         517      517
Impacted Files Coverage 螖
openml/runs/functions.py 82.32% <100%> (+0.09%) 猬嗭笍
openml/datasets/functions.py 95.51% <100%> (+0.02%) 猬嗭笍
openml/evaluations/functions.py 92.15% <100%> (+0.15%) 猬嗭笍

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 f461732...c6eb05f. Read the comment docs.

@mfeurer mfeurer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Amazingly simple, I really like this example!

Only a few very minor comments.

Comment thread examples/40_paper/2018_ida_strang_example.py Outdated
Comment thread examples/40_paper/2018_ida_strang_example.py Outdated
Comment thread examples/40_paper/2018_ida_strang_example.py Outdated
Comment thread openml/evaluations/functions.py
Comment thread openml/runs/functions.py
@janvanrijn
janvanrijn requested a review from mfeurer October 2, 2019 15:30
@mfeurer
mfeurer merged commit 5a2830c into develop Oct 2, 2019
@mfeurer
mfeurer deleted the add_example_strang branch October 2, 2019 16:17
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.

3 participants