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] OWTestLearners: Fix reporting results table #1421

Merged
merged 4 commits into from
Jul 8, 2016

Conversation

kernc
Copy link
Contributor

@kernc kernc commented Jul 7, 2016

No description provided.

@codecov-io
Copy link

codecov-io commented Jul 7, 2016

Current coverage is 87.94%

Merging #1421 into master will decrease coverage by 0.03%

@@             master      #1421   diff @@
==========================================
  Files            77         77          
  Lines          7582       7582          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           6671       6668     -3   
- Misses          911        914     +3   
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by b77c021...2b8cad3

kernc added 3 commits July 8, 2016 13:55
When the returned ndarray's values were ints, it resulted in an
odd issue where 1 / Discrete => Discrete whereas
1. / Discrete => np.float64.
Before, self.attribute() returned TREE_UNDEFINED (-2) for leaf nodes
which failed with domains of a single feature and worked incorrectly
with all other domains (it took the second to last feature).
Now, use self.parent.attribute() to format the value of the split.
@lanzagar lanzagar merged commit f18f6a6 into biolab:master Jul 8, 2016
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.

3 participants