Skip to content

add string representation for runs - #391

Merged
mfeurer merged 5 commits into
openml:developfrom
amueller:run_str
Mar 28, 2018
Merged

add string representation for runs#391
mfeurer merged 5 commits into
openml:developfrom
amueller:run_str

Conversation

@amueller

Copy link
Copy Markdown
Contributor

Didn't add a test... but tests are failing anyway? What's up with that?

@amueller

Copy link
Copy Markdown
ContributorAuthor

part of #75

@mfeurer

Copy link
Copy Markdown
Collaborator

I think tests on the development branch fail in general right now.

@joaquinvanschoren

joaquinvanschoren commented Feb 7, 2018 via email

Copy link
Copy Markdown
Contributor

@mfeurer

Copy link
Copy Markdown
Collaborator

This dataset is one of the culprits. @janvanrijn will fix issues with the test server within the next few days.

@codecov-io

codecov-io commented Feb 19, 2018

Copy link
Copy Markdown

Codecov Report

Merging #391 into develop will decrease coverage by 0.15%.
The diff coverage is 37.5%.

Impacted file tree graph

@@ Coverage Diff @@## develop #391 +/- ##
===========================================
- Coverage 89.53% 89.37% -0.16% 
===========================================
Files 32 32 Lines 2676 2683 +7 ===========================================
+ Hits 2396 2398 +2 - Misses 280 285 +5
Impacted FilesCoverage Δ
openml/runs/run.py90.71% <37.5%> (-1.9%)⬇️

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 2513667...e090d36. Read the comment docs.

@mfeurer
mfeurer merged commit 5058e1d into openml:developMar 28, 2018
Sign up for freeto 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.

5 participants

@amueller@mfeurer@joaquinvanschoren@codecov-io@janvanrijn