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?
---