From 64c8f6fa6d01e284143f6cdaab3ade163924ceca Mon Sep 17 00:00:00 2001 From: Philip Cho Date: Mon, 10 Jul 2017 09:35:48 -0700 Subject: [PATCH] Use old parallel algorithm for histogram construction by default (#2501) It has been reported that new parallel algorithm (#2493) results in excessive message usage (see issue #2326). Until issues are resolved, XGBoost should use the old parallel algorithm by default. The user would have to specify `enable_feature_grouping=1` manually to enable the new algorithm. --- src/tree/fast_hist_param.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/tree/fast_hist_param.h b/src/tree/fast_hist_param.h index 7bd3be9f2..5ca9e0b5e 100644 --- a/src/tree/fast_hist_param.h +++ b/src/tree/fast_hist_param.h @@ -42,7 +42,7 @@ struct FastHistParam : public dmlc::Parameter { "advanced use"); DMLC_DECLARE_FIELD(sparse_threshold).set_range(0, 1.0).set_default(0.2) .describe("percentage threshold for treating a feature as sparse"); - DMLC_DECLARE_FIELD(enable_feature_grouping).set_lower_bound(0).set_default(1) + DMLC_DECLARE_FIELD(enable_feature_grouping).set_lower_bound(0).set_default(0) .describe("if >0, enable feature grouping to ameliorate work imbalance " "among worker threads"); DMLC_DECLARE_FIELD(max_conflict_rate).set_range(0, 1.0).set_default(0)