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
[DOC] Regression api #3751
[DOC] Regression api #3751
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
API changes are done in the style of #3742. Should probably wait for that first, but I approve of the changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small bits.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a push fixing the module import, both base classes now show on the page.
What does this implement/fix? Explain your changes.
This adds CNNRegressor, TapNetRegressor and RocketRegressor to the regression api. I have also expanded the docstring for TapNetRegressor to be a bit more descriptive and added and init to the regression.deep_learning package. CNNRegressor and TapNetRegressor should be private, but that would require deprecation, and is beyond the scope of this [DOC] PR.