Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

MAINT bump required liac-arff version#427

Merged
mfeurer merged 2 commits intodevelopopenml/openml-python:developfrom
maint/bump_liac_arff_dependencyopenml/openml-python:maint/bump_liac_arff_dependencyCopy head branch name to clipboard
Mar 29, 2018
Merged

MAINT bump required liac-arff version#427
mfeurer merged 2 commits intodevelopopenml/openml-python:developfrom
maint/bump_liac_arff_dependencyopenml/openml-python:maint/bump_liac_arff_dependencyCopy head branch name to clipboard

Conversation

@mfeurer
Copy link
Collaborator

@mfeurer mfeurer commented Mar 29, 2018

New version is more resilient with respect to commas in data. However, it cannot yet read all commas, that's for the future.

@mfeurer mfeurer requested a review from janvanrijn March 29, 2018 08:32
@codecov-io
Copy link

codecov-io commented Mar 29, 2018

Codecov Report

Merging #427 into develop will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop    #427      +/-   ##
==========================================
+ Coverage    89.37%   89.5%   +0.12%     
==========================================
  Files           32      32              
  Lines         2683    2744      +61     
==========================================
+ Hits          2398    2456      +58     
- Misses         285     288       +3
Impacted Files Coverage 螖
openml/datasets/dataset.py 81.01% <0%> (+3.36%) 猬嗭笍

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 5b701bb...e789e5a. Read the comment docs.

Copy link
Member

@janvanrijn janvanrijn left a comment

Choose a reason for hiding this comment

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

Seems like you're also maintaining the liac-arff library

@mfeurer
Copy link
Collaborator Author

mfeurer commented Mar 29, 2018

Yep...

@mfeurer mfeurer merged commit fb20762 into develop Mar 29, 2018
@mfeurer mfeurer deleted the maint/bump_liac_arff_dependency branch April 23, 2018 07:09
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.