Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Example section of Readme.md #116

Merged
merged 3 commits into from
Mar 19, 2016
Merged

Conversation

pronojitsaha
Copy link
Contributor

@rhiever Readme.md has the tpot_mnist_pipeline.py (in the Example section) which is of a very early version. We have updated the tpot_mnist_pipeline.py in the tutorials section..but not done so in the Example of Readme.md. So I have updated the same including changing StratifiedShuffleSplit to train_test_split as discussed in #114

@coveralls
Copy link

Coverage Status

Coverage remained the same at 28.741% when pulling d4a9dcc on pronojitsaha:master into 6860b92 on rhiever:master.

@rhiever
Copy link
Contributor

rhiever commented Mar 19, 2016

I think all the extra models makes the example code look too complex. How about we just have the example code be a generated Random Forest call with 100+ estimators? That's usually what comes out of TPOT for the MNIST data set now anyway.

@pronojitsaha
Copy link
Contributor Author

Yes, I was also thinking about the complexity to understand this for a first time user. I will look into it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 28.741% when pulling d47e96b on pronojitsaha:master into 6860b92 on rhiever:master.

rhiever pushed a commit that referenced this pull request Mar 19, 2016
Update Example section of Readme.md
@rhiever rhiever merged commit 524bd69 into EpistasisLab:master Mar 19, 2016
@rhiever
Copy link
Contributor

rhiever commented Mar 19, 2016

Good to go! Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants