Skip to content

[FIX] Rank: Fix crash on dataset with missing values#3458

Merged
lanzagar merged 1 commit intobiolab:masterfrom
VesnaT:fix_rank
Dec 10, 2018
Merged

[FIX] Rank: Fix crash on dataset with missing values#3458
lanzagar merged 1 commit intobiolab:masterfrom
VesnaT:fix_rank

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Dec 8, 2018

Issue

OWRank crashed when input dataset with missing values (i.e. testing_dataset_cls, testing_dataset_reg).

Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Dec 8, 2018

Codecov Report

Merging #3458 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3458      +/-   ##
==========================================
+ Coverage   83.25%   83.27%   +0.01%     
==========================================
  Files         365      365              
  Lines       64300    64315      +15     
==========================================
+ Hits        53534    53558      +24     
+ Misses      10766    10757       -9

@lanzagar lanzagar merged commit 30a33ab into biolab:master Dec 10, 2018
@ajdapretnar
Copy link
Contributor

I can't reproduce the error with any regular data with missing values. I also can't find the mentioned datasets. Could you please tell me how to test this so I can merge? :)

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