-
-
Notifications
You must be signed in to change notification settings - Fork 26.2k
[MRG+1] Add MLP to what's new #5828
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
[MRG+1] Add MLP to what's new #5828
Conversation
@@ -25,6 +25,9 @@ New features | |||
- Added the :class:`ensemble.IsolationForest` class for anomaly detection based on | |||
random forests. By `Nicolas Goix`_. | |||
|
|||
- Added new supervised learning algorithm: :ref: `Multi-layer Perceptron <multi-layer-perceptron>` | |||
by `Issam H. Laradji`_ |
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.
you can also add a link to the issue as in some other places in whatsnew.
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.
Added the link to relevant PR.
b22d475
to
fe32105
Compare
@@ -25,6 +25,9 @@ New features | |||
- Added the :class:`ensemble.IsolationForest` class for anomaly detection based on | |||
random forests. By `Nicolas Goix`_. | |||
|
|||
- Added new supervised learning algorithm: :ref:`Multi-layer Perceptron <multi-layer-perceptron>` |
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.
the correct target is multilayer_perceptron
, which is why the linking doesn't work. See https://raw.githubusercontent.com/scikit-learn/scikit-learn/master/doc/modules/neural_networks_supervised.rst
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 did try that target before but used dash instead of underscore, fixed it now. Thanks for the help 😃
fe32105
to
bab2233
Compare
LGTM. +1 for merge. |
LGTM, thanks :) |
[MRG+1] Add MLP to what's new
Fixes #5640, updated what's new.