madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From iyerr3 <...@git.apache.org>
Subject [GitHub] madlib pull request #289: RF: Add impurity variable importance
Date Wed, 11 Jul 2018 15:10:46 GMT
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. 


---

Mime
View raw message