Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] Confusion Matrix: do not append extra column if empty #2386

Merged
merged 3 commits into from
Jun 7, 2017
Merged

[FIX] Confusion Matrix: do not append extra column if empty #2386

merged 3 commits into from
Jun 7, 2017

Conversation

jerneju
Copy link
Contributor

@jerneju jerneju commented Jun 7, 2017

Issue

screenshot_20170607_111147

Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@jerneju
Copy link
Contributor Author

jerneju commented Jun 7, 2017

@codecov-io
Copy link

codecov-io commented Jun 7, 2017

Codecov Report

Merging #2386 into master will increase coverage by 0.01%.
The diff coverage is 86.36%.

@@            Coverage Diff            @@
##           master   #2386      +/-   ##
=========================================
+ Coverage   73.29%   73.3%   +0.01%     
=========================================
  Files         317     317              
  Lines       55512   55519       +7     
=========================================
+ Hits        40687   40698      +11     
+ Misses      14825   14821       -4

@kernc
Copy link
Contributor

kernc commented Jun 7, 2017

I'll merge this PR for you if you rid it of the third commit. 😸

Copy link
Contributor

@kernc kernc left a comment

Choose a reason for hiding this comment

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

Much thanks for switching commits order!

@kernc kernc merged commit 14d9fd7 into biolab:master Jun 7, 2017
@jerneju jerneju deleted the index-confusionmatrix branch June 8, 2017 06:58
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