Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] owpreprocess: Handle columns with only NaN values #2064

Merged
merged 1 commit into from
Mar 3, 2017
Merged

[FIX] owpreprocess: Handle columns with only NaN values #2064

merged 1 commit into from
Mar 3, 2017

Conversation

jerneju
Copy link
Contributor

@jerneju jerneju commented Mar 2, 2017

Issue

ZeroDivisonError - Weights sum to zero, can't be normalized
In case when all rows in a column are NaN then it throws that error.
https://sentry.io/biolab/orange3/issues/227119466/

Description of changes

Add another except in try block and now it shows an error message in the widget.

Includes
  • Code changes
  • Tests
  • Documentation

@jerneju
Copy link
Contributor Author

jerneju commented Mar 2, 2017

@codecov-io
Copy link

codecov-io commented Mar 2, 2017

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2064      +/-   ##
==========================================
+ Coverage   69.81%   69.82%   +<.01%     
==========================================
  Files         315      315              
  Lines       53957    53962       +5     
==========================================
+ Hits        37670    37678       +8     
+ Misses      16287    16284       -3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05be958...08c7255. Read the comment docs.

@jerneju
Copy link
Contributor Author

jerneju commented Mar 2, 2017

In case when all rows in a column are NaN then it throws that error.
GH-2064
"""
table = Table(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the same dataset is used in multiple tests, would you consider moving it to datasets?

@lanzagar lanzagar self-assigned this Mar 3, 2017
ZeroDivisonError - Weights sum to zero, can't be normalized
In case when all rows in a column are NaN then it throws that error.
https://sentry.io/biolab/orange3/issues/227119466/

Add another except in try block and now it shows an error message in the widget.
@lanzagar lanzagar changed the title [FIX] ZeroDivisionError - owpreprocess [FIX] owpreprocess: Handle columns with only NaN values Mar 3, 2017
@lanzagar lanzagar merged commit 34b5e94 into biolab:master Mar 3, 2017
@jerneju jerneju deleted the zerodivisionerror-owpreprocess branch April 20, 2017 13:47
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