Skip to content

[FIX] stats.utils: Don't count zeros as nans#6314

Merged
markotoplak merged 1 commit intobiolab:masterfrom
janezd:utils-stat-sparse-nans
Jan 27, 2023
Merged

[FIX] stats.utils: Don't count zeros as nans#6314
markotoplak merged 1 commit intobiolab:masterfrom
janezd:utils-stat-sparse-nans

Conversation

@janezd
Copy link
Contributor

@janezd janezd commented Jan 25, 2023

Issue

Fixes #6074.

Includes
  • Code changes
  • Tests

@janezd janezd force-pushed the utils-stat-sparse-nans branch 3 times, most recently from f6690cd to 7af9325 Compare January 26, 2023 11:32
@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Merging #6314 (39cdce6) into master (a25a9d2) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6314      +/-   ##
==========================================
- Coverage   87.48%   87.48%   -0.01%     
==========================================
  Files         316      316              
  Lines       67968    67965       -3     
==========================================
- Hits        59459    59456       -3     
  Misses       8509     8509              

@markotoplak markotoplak changed the title stats.utils: Don't count zeros as nans [FIX] stats.utils: Don't count zeros as nans Jan 27, 2023
@markotoplak markotoplak merged commit df1cdb5 into biolab:master Jan 27, 2023
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.

Orange.statistics.utils.stats() counts defined values instead of NaNs

2 participants