-
Notifications
You must be signed in to change notification settings - Fork 20
end-to-end example of UCI Dataset #46
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
Conversation
Hello! thanks for this, it's a good start. I would suggest:
yes plots would be very welcome too, thanks 👍 PS: the ZScoreTransform is not available; you can either add it directly to MLJModels or code it yourself or use another available transformer |
Readme changes are all fine by me, thanks! |
@tlienart I have added the comparison of DecisionTreeRegressor with RandomForestRegressor and have performed hyperparameter tuning too. I have made use of Standardizer and also plotted a heatmap. |
Thanks a lot for your contribution @ashryaagr , looks good to me at first glance, I'll do another full depth pass within the next few days. Cheers! |
Sorry for the delay, there seems to be a bug in how CSV reads requests like this, the line to recover the data doesn't seem to work anymore. |
nvm, it's an issue with CSV, fixed by upgrading. |
ok that looks good, I've done a few fixes/changes and added your name at the top (thanks!!) and will merge this. I'll wait for https://travis-ci.com/github/alan-turing-institute/MLJTutorials/builds/160590950 to pass first. At least locally, updating everything, it works. so should be good |
closing in favour of #51 , thanks! |
Thanks for this both @tlienart @ashryaagr ! 💪 |
Initial start for the UCI Dataset Airfoil. I will further add code for plotting and scaling the input using ZScoreTransform from ScikitLearn.jl.