Skip to content

[FIX] OWDataProjectionWidget: Consider tables with nan-s equal#3435

Merged
lanzagar merged 1 commit intobiolab:masterfrom
VesnaT:fix_widget
Dec 3, 2018
Merged

[FIX] OWDataProjectionWidget: Consider tables with nan-s equal#3435
lanzagar merged 1 commit intobiolab:masterfrom
VesnaT:fix_widget

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Dec 3, 2018

Issue

When widget received (unchanged) data with missing values on the input (i.e. Reloading the file) the embedding was recalculated.

Description of changes

When comparing previous and current table.X, consider np.nan-s equal.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Dec 3, 2018

Codecov Report

Merging #3435 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3435      +/-   ##
==========================================
+ Coverage   83.22%   83.22%   +<.01%     
==========================================
  Files         362      362              
  Lines       64105    64106       +1     
==========================================
+ Hits        53350    53354       +4     
+ Misses      10755    10752       -3

@VesnaT VesnaT changed the title OWDataProjectionWidget: Consider tables with nan-s equal [FIX]OWDataProjectionWidget: Consider tables with nan-s equal Dec 3, 2018
@lanzagar lanzagar changed the title [FIX]OWDataProjectionWidget: Consider tables with nan-s equal [FIX] OWDataProjectionWidget: Consider tables with nan-s equal Dec 3, 2018
@lanzagar lanzagar merged commit 323a738 into biolab:master Dec 3, 2018
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.

2 participants