-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
DOC Add var_ attribute and deprecate sigma_ in GaussianNB #18842
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 Add var_ attribute and deprecate sigma_ in GaussianNB #18842
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.
Please add an entry to the change log at doc/whats_new/v*.rst
. Like the other entries there, please reference this pull request with :pr:
and credit yourself (and other contributors if applicable) with :user:
.
Which version should the whatsnew be in? |
`1.0.rst`. You might need to merge master into your branch.
…On Mon, 21 Dec 2020 at 12:12, Shao Yang Hong ***@***.***> wrote:
Which version should the whatsnew be in?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#18842 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABY32P3DDXHR2QHONQU5N5LSV4URTANCNFSM4TWBO2VQ>
.
--
Guillaume Lemaitre
Scikit-learn @ Inria Foundation
https://glemaitre.github.io/
|
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.
0.24 has already been released. Please change 0.24 and 1.1 to 1.0 and 1.2.
Otherwise lgtm.
…deprecatesigma # Conflicts: # doc/whats_new/v1.0.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.
LGTM, thanks @hongshaoyang. Just 2 minor comments
Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
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.
Please add an entry to the change log at doc/whats_new/v*.rst
. Like the other entries there, please reference this pull request with :pr:
and credit yourself (and other contributors if applicable) with :user:
.
Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
@glemaitre Whatsnew has been added. Code changes applied. |
I just solved the conflict in whats new. Merging. Thanks @hongshaoyang |
Reference Issues/PRs
Closes #16760
What does this implement/fix? Explain your changes.
sigma
means std deviation which is confusing asGaussianNB.sigma_
is currently the variance of each feature per classRename sigma_ to var_ and deprecate sigma_
Any other comments?