Skip to content

Commit

Permalink
[backport] Delay the check for vector leaf. (#9509) (#9533)
Browse files Browse the repository at this point in the history
  • Loading branch information
trivialfis authored Aug 29, 2023
1 parent a0d3573 commit add57f8
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions src/gbm/gbtree.cc
Original file line number Diff line number Diff line change
Expand Up @@ -146,14 +146,6 @@ void GBTree::Configure(Args const& cfg) {
if (specified_updater_) {
error::WarnManualUpdater();
}

if (model_.learner_model_param->IsVectorLeaf()) {
CHECK(tparam_.tree_method == TreeMethod::kHist || tparam_.tree_method == TreeMethod::kAuto)
<< "Only the hist tree method is supported for building multi-target trees with vector "
"leaf.";
CHECK(ctx_->IsCPU()) << "GPU is not yet supported for vector leaf.";
}

LOG(DEBUG) << "Using tree method: " << static_cast<int>(tparam_.tree_method);

if (!specified_updater_) {
Expand Down Expand Up @@ -225,6 +217,13 @@ void GBTree::UpdateTreeLeaf(DMatrix const* p_fmat, HostDeviceVector<float> const

void GBTree::DoBoost(DMatrix* p_fmat, HostDeviceVector<GradientPair>* in_gpair,
PredictionCacheEntry* predt, ObjFunction const* obj) {
if (model_.learner_model_param->IsVectorLeaf()) {
CHECK(tparam_.tree_method == TreeMethod::kHist || tparam_.tree_method == TreeMethod::kAuto)
<< "Only the hist tree method is supported for building multi-target trees with vector "
"leaf.";
CHECK(ctx_->IsCPU()) << "GPU is not yet supported for vector leaf.";
}

TreesOneIter new_trees;
bst_target_t const n_groups = model_.learner_model_param->OutputLength();
monitor_.Start("BoostNewTrees");
Expand Down

0 comments on commit add57f8

Please sign in to comment.