add custom regressor classifer #1186#1191
add custom regressor classifer #1186#1191ankitrajixr wants to merge 4 commits intoEpistasisLab:master-archivedfrom
Conversation
|
Hi @ankitrajixr , thanks for submitting this pull request. There are a couple of issues here:
|
|
@ankitrajixr as another note, please also make sure to include detail describing the exact nature of the changes this pull request (when completed) will introduce. For reference, you may look at the following blog post: https://github.blog/2015-01-21-how-to-write-the-perfect-pull-request/ |
|
Note that this PR will fail to pass checks and the proposed regressor will not function due to its use of the kernels within the configuration dictionary, which will cause TPOT to crash when trying to evaluate any pipeline string with these kernels due to them not existing in TPOT's operator space as TPOT does not consider parameters when importing operators. #1218 touches on some of the changes that would need to be made to TPOT to support these kind of configurations (needing to either allow the user to pass references to objects used as parameters when instantiating TPOT or finding a way to dynamically assign those references within TPOT itself). Until those changes are made, I don't think this PR can be merged quite yet. |
#1186
What does this PR do?
We found certain regressors and classifiers not included in the model selection of the machine learning pipeline. Hence, we have included a few.
Contributors are:
Ankit Raj https://github.com/ankitrajixr
Darshan Chandra https://github.com/drshnchndr
Sindhu Subramanya https://github.com/Sindhu-Subramanya