sivarajankumar / rf-ace

Automatically exported from code.google.com/p/rf-ace
0 stars 0 forks source link

Seed parameter not needed for prediction #28

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What version of the product are you using? On what operating system?
rf-ace-predict-*.exe
Every OS

Please provide any additional information below.
The seed parameter should only play a role for building a classifier, but not 
for prediction. Hence, to avoid confusion it should be removed from both the 
interface as well as the command line feedback.

best regards,
Berni

Original issue reported on code.google.com by berni.le...@gmail.com on 30 Mar 2012 at 7:28

GoogleCodeExporter commented 8 years ago
Actually it is needed at the moment used. The algorithm flips a biased coin 
when it encounters a NaN to split on. Bias is determined based on the fraction 
of training data on the left and right branches.

Original comment by timo.erk...@gmail.com on 29 May 2012 at 5:31