Github user iyerr3 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/289#discussion_r201728550
--- Diff: src/modules/recursive_partitioning/DT_impl.hpp ---
@@ -1512,6 +1512,9 @@ DecisionTree<Container>::computeVariableImportance(
uint64_t maj_count = getMajorityCount(node_index);
uint64_t min_count = node_count - maj_count;
+ if (min_count == 0) {
--- End diff --
I've pushed a commit to master that fixes the need to do this hack. Please
rebase on master and remove these lines.
---
