Skip to content

Conversation

@ArlindKadra
Copy link
Member

Fix ascii decoding error with python 2.

@ArlindKadra ArlindKadra requested a review from mfeurer March 13, 2018 10:50
@mfeurer
Copy link
Collaborator

mfeurer commented Mar 14, 2018

Please also write a unit test following the example given in #378.

@codecov-io
Copy link

codecov-io commented Mar 14, 2018

Codecov Report

Merging #418 into develop will decrease coverage by 0.14%.
The diff coverage is 68.75%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #418      +/-   ##
===========================================
- Coverage     89.1%   88.95%   -0.15%     
===========================================
  Files           32       32              
  Lines         2671     2681      +10     
===========================================
+ Hits          2380     2385       +5     
- Misses         291      296       +5
Impacted Files Coverage Δ
openml/tasks/split.py 91.08% <66.66%> (-2.67%) ⬇️
openml/datasets/dataset.py 77.64% <66.66%> (-0.22%) ⬇️
openml/datasets/data_feature.py 69.56% <75%> (-0.44%) ⬇️

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 f4387d6...7ef82d8. Read the comment docs.

@mfeurer mfeurer merged commit deb769e into develop Mar 14, 2018
@ArlindKadra ArlindKadra deleted the fix378 branch March 19, 2018 11:11
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.

4 participants