biolab / orange3-explain

Explainable AI add-on for Orange3
GNU General Public License v3.0
6 stars 5 forks source link

[FIX] Feature Importance: Handle missing target #25

Closed VesnaT closed 3 years ago

VesnaT commented 3 years ago
Issue

The Feature Importance widget displays an uninformative error message, when data has missing values in target variable.

Description of changes

Remove instances with missing class variables and show more informative message, i.e.: Instances with unknown target values were removed from data.

Includes
codecov-commenter commented 3 years ago

Codecov Report

Merging #25 (a76b16a) into master (12bec08) will increase coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   95.59%   95.62%   +0.03%     
==========================================
  Files           7        7              
  Lines        1817     1830      +13     
==========================================
+ Hits         1737     1750      +13     
  Misses         80       80              
Impacted Files Coverage Δ
...ngecontrib/explain/widgets/owexplainfeaturebase.py 97.97% <100.00%> (+0.02%) :arrow_up:
...contrib/explain/widgets/owpermutationimportance.py 91.04% <100.00%> (+0.27%) :arrow_up:

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 12bec08...a76b16a. Read the comment docs.