Add TODO about FastConfig naming needing to be updated

As discussed here: microsoft#6024 (comment)
Ten0 · Jan 12, 2024 · 993f8f5 · 993f8f5
1 parent c7e9d2e
commit 993f8f5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/c_api.cpp b/src/c_api.cpp
@@ -2113,8 +2113,8 @@ int LGBM_BoosterCalcNumPredict(BoosterHandle handle,
   API_END();
 }
 
-
-
+// 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.
 int LGBM_FastConfigFree(FastConfigHandle fastConfig) {
   API_BEGIN();
   delete reinterpret_cast<SingleRowPredictor*>(fastConfig);