Use 8 estimators for small and medium KumoTabular in benchmarks - #991
Conversation
The size options from #983 set 16 estimators for every size; large keeps 16. Signed-off-by: Jingang Qu <jqu@nvidia.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA/structured-data-models/.coderabbit.yaml Review profile: QUIET Plan: Enterprise Run ID: 📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe medium model default and small and medium benchmark configurations now use 8 estimators instead of 16. The large configuration and other described settings are unchanged. ChangesKumoTabular estimator count
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~4 minutes Merge Risk: ⚪ Minimal · up to The estimator defaults and benchmark configurations match the stated size-specific counts. No merge-blocking issue was identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
#983 raised every size to 16 estimators. This PR corrects it and sets 8 estimators by default for small and medium
KumoTabularin the TabArena/BeyondArena, ScoringBench and TALENT benchmarks. Large keeps 16.