search

sanity / quickml

A fast and easy to use decision tree learner in java
http://quickml.org/
GNU Lesser General Public License v3.0
232 stars 52 forks source link

Documentation on quickml.org is possibly outdated #133

Closed tarjeir closed 9 years ago

tarjeir commented 9 years ago

This line: final RandomForest randomForest = new RandomForestBuilder().buildPredictiveModel(irisDataset);

RandomForestBuilder is an abstract class in 0.9.4

Should probably be replaced by something like this (or simpler):

final RandomDecisionForest randomForest =new RandomDecisionForestBuilder<>(new DecisionTreeBuilder<>() .scorerFactory(new GRPenalizedGiniImpurityScorerFactory(1.0)) .maxDepth(8) .minAttributeValueOccurences(0) .attributeIgnoringStrategy(new IgnoreAttributesWithConstantProbability(0.2))) .numTrees(2) .buildPredictiveModel(irisDataset);

Tarjei

athawk81 commented 9 years ago

which file has this problem? Thanks for your help btw.

tarjeir commented 9 years ago

Sorry for not stating that 100% clear: It is the quickml.org front page.

BR

Tarjei

sanity commented 9 years ago

@tarjeir You are very welcome to submit a pull request, the site is maintained in the gh-pages branch. I'm currently on the road so it may be a few days before I can fix this.

athawk81 commented 9 years ago

just pushed effectively the change Tarjei requested.