Github user iyerr3 commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/289#discussion_r201490136
  
    --- Diff: src/modules/recursive_partitioning/decision_tree.cpp ---
    @@ -502,10 +502,13 @@ get_variable_importance::run(AnyType &args){
        ColumnVector combined_var_imp(n_cat_features + n_con_features);
        combined_var_imp << cat_var_importance, con_var_importance;
     
    -    // Avoid divide by zero by adding a small number.
    +    // Avoid divide by zero by replacing with a small number if necessary
    --- End diff --
    
    Out of curiosity: what was wrong with the previous method? 


---

Reply via email to