search

ThrunGroup / FastForest

7 stars 0 forks source link

Implement XGBoostClassifier #72

Open motiwari opened 2 years ago

motiwari commented 2 years ago

We should set this up as a baseline.

It may be valuable to have this as a separate class that inherits Tree or TreeClassifier.

It should take parameters like the boosting learning rate etc.

motiwari commented 2 years ago

Just to be clear, only the boosting part of XGBoost is necessary. We don't need any of the systems-level optimization, just the algorithm

RyanK0902 commented 2 years ago

When I implement XGBoost with their package, it runs all the systems-level optimizations by default.

motiwari commented 2 years ago

Based on discussions with Martin, this is now low pri in favor of #135 and #134