Random forest can now take user defined parameters.

manifoldai / merf

Mixed Effects Random Forest

MIT License

219 stars 52 forks source link

Random forest can now take user defined parameters. #25

Closed arose13 closed 5 years ago

arose13 commented 5 years ago

The current implementation of MERF only allows the user to set the number of trees in the random forest. This pull request allows the user to set any parameter they want but prevents them from changing important params like having the out of bag score on.

I hope you accept this it has been useful for me.

arose13 commented 5 years ago

Hey, I am absolutely not clear what is wrong with the CI tests when I'm able to pass on tests on my local machine.

7 of 7 tests passed in 8.044s

OK

resdntalien commented 5 years ago

So sorry for the lateness of this. Will take a look at this and try to get it merged in!

resdntalien commented 5 years ago

Look like the CI failing is a Lint issue