random-forest icon indicating copy to clipboard operation
random-forest copied to clipboard

Feature Bagging Options

Open micha-lmxt opened this issue 5 years ago • 1 comments

Hi,

thank you for your work on this library.

Other implementations of random forests apply feature bagging at every split during tree generation. So I think it would be better to implement it at the CART level, not here.

Also, "replacement" is a bit of weird option, since it makes no sense to supply a feature more than once to the tree generation. Thus, it basically just randomly shrinks the number of features used for a tree.

micha-lmxt avatar Oct 12 '20 10:10 micha-lmxt

Thanks for your contributions. Please could you fix the test cases that are currently failing ?

lpatiny avatar Oct 15 '20 07:10 lpatiny