Skip to content

Remove unnecessary writes - #323

Merged
amueller merged 1 commit into
developfrom
fix_#79
Oct 9, 2017
Merged

amueller merged 1 commit into
developfrom
fix_#79

Conversation

@mfeurer

@mfeurer mfeurer commented Oct 8, 2017 •

Copy link
Copy Markdown
Collaborator

Fixes #79

@codecov-io

codecov-io commented Oct 8, 2017 •

Copy link
Copy Markdown

Codecov Report

Merging #323 into develop will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #323      +/-   ##
===========================================
- Coverage    89.74%   89.72%   -0.02%     
===========================================
  Files           32       32              
  Lines         2516     2512       -4     
===========================================
- Hits          2258     2254       -4     
  Misses         258      258
Impacted Files Coverage Δ
openml/datasets/functions.py 90.64% <ø> (-0.1%) ⬇️
openml/runs/functions.py 87.89% <ø> (-0.06%) ⬇️

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 9eedb90...c28258a. Read the comment docs.

@amueller amueller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

look good

@amueller amueller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good.

@amueller
amueller merged commit 2448826 into develop Oct 9, 2017
@mfeurer
mfeurer deleted the fix_#79 branch October 9, 2017 15:41
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