summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2024-05-03 17:33:00 +0200
committerGitHub <noreply@github.com>2024-05-03 17:33:00 +0200
commitac654e7eb268e5af6c1a1e9eb5c4726b61b7e4a6 (patch)
tree6f84fbb6e2faad999e2507aef2fe54992b7f39af
parentdf5586db4480974e5fb9e1d10117b7c80c9c40ee (diff)
parent18adc4f729c5c0afe993e1bee13e971959031fa6 (diff)
Merge pull request #31104 from vespa-engine/havardpe/upper-limit-on-total-size-of-compiled-gbdt-models
Avoid compiling gbdt models with many conditions
-rw-r--r--eval/src/vespa/eval/eval/gbdt.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/eval/src/vespa/eval/eval/gbdt.cpp b/eval/src/vespa/eval/eval/gbdt.cpp
index 3422228b03c..7ab4c4ae822 100644
--- a/eval/src/vespa/eval/eval/gbdt.cpp
+++ b/eval/src/vespa/eval/eval/gbdt.cpp
@@ -154,6 +154,9 @@ Optimize::select_best(const ForestStats &stats,
if ((stats.tree_sizes.back().size > 12) && (path_len > 2500.0)) {
return apply_chain(VMForest::optimize_chain, stats, trees);
}
+ if (stats.total_size > 25000) {
+ return apply_chain(VMForest::optimize_chain, stats, trees);
+ }
return Optimize::Result();
}