Skip to content

[FIX] Confusion Matrix: Show error on regression results#1709

Merged
lanzagar merged 1 commit intobiolab:masterfrom
janezd:fix-confusion-matrix-regression
Nov 4, 2016
Merged

[FIX] Confusion Matrix: Show error on regression results#1709
lanzagar merged 1 commit intobiolab:masterfrom
janezd:fix-confusion-matrix-regression

Conversation

@janezd
Copy link
Copy Markdown
Contributor

@janezd janezd commented Nov 4, 2016

Issue

Confusion matrix set a warning (instead of error) when it received regression data. After that, it tried to use it and crashed.

Description of changes

The widget now shows an error and proceeds as if the data was None (e.g. clears the table if shown, disables the report button etc.)

Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 4, 2016

Current coverage is 88.86% (diff: 100%)

Merging #1709 into master will not change coverage

@@             master      #1709   diff @@
==========================================
  Files            82         82          
  Lines          8802       8802          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           7822       7822          
  Misses          980        980          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 3d865bc...20336ee

@lanzagar lanzagar merged commit 1a35510 into biolab:master Nov 4, 2016
@astaric astaric modified the milestone: 3.3.9 Nov 28, 2016
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