Skip to content

[FIX] Distance Matrix: Fix crash on numeric meta vars as labels#5664

Merged
ajdapretnar merged 1 commit intobiolab:masterfrom
janezd:dist-matrix-meta-label
Nov 3, 2021
Merged

[FIX] Distance Matrix: Fix crash on numeric meta vars as labels#5664
ajdapretnar merged 1 commit intobiolab:masterfrom
janezd:dist-matrix-meta-label

Conversation

@janezd
Copy link
Contributor

@janezd janezd commented Oct 22, 2021

Issue

Fixes #5651.

If meta attributes contain a numeric and string variable, meta variables have dtype np.object.

Description of changes

If numeric variable is chosen as a label, its values must be cast to float before mapping to colors.

Includes
  • Code changes
  • Tests

@codecov
Copy link

codecov bot commented Oct 22, 2021

Codecov Report

Merging #5664 (7212f33) into master (45f17d5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5664   +/-   ##
=======================================
  Coverage   86.01%   86.01%           
=======================================
  Files         315      315           
  Lines       65948    65950    +2     
=======================================
+ Hits        56726    56730    +4     
+ Misses       9222     9220    -2     

@ajdapretnar ajdapretnar merged commit 957b12b into biolab:master Nov 3, 2021
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.

Distance Matrix: crash on numeric meta

2 participants