From 03ebb8880cc3ee9e5c1ac580fd72ca44039aee61 Mon Sep 17 00:00:00 2001 From: Thomas BESSOU Date: Fri, 12 Jan 2024 01:27:13 +0100 Subject: [PATCH] add more cleanup comment --- src/c_api.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/c_api.cpp b/src/c_api.cpp index 8562815432ef..d7a2ea6a12b5 100644 --- a/src/c_api.cpp +++ b/src/c_api.cpp @@ -2121,6 +2121,8 @@ int LGBM_BoosterCalcNumPredict(BoosterHandle handle, // TODO in future versions of LightGBM, public API named around `FastConfig` should be made named around // `SingleRowPredictor`, because it is specific to single row prediction, and doesn't actually hold only config. +// At the same time, one should consider removing the old non-fast single row public API that stores its Predictor +// in the booster, because that will enable removing these Predictors from the Booster, and associated initialization. int LGBM_FastConfigFree(FastConfigHandle fastConfig) { API_BEGIN(); delete reinterpret_cast(fastConfig);