10000 BernoulliNB and MultinomialNB documentation for alpha=0 · Issue #10772 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

BernoulliNB and MultinomialNB documentation for alpha=0 #10772

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.

Sign up for GitHub

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

Jump to bottom
Closed
rmalouf opened this issue Mar 7, 2018 · 6 comments · Fixed by SkuaD01/scikit-learn#2 or #22269
Closed

BernoulliNB and MultinomialNB documentation for alpha=0 #10772

rmalouf opened this issue Mar 7, 2018 · 6 comments · Fixed by SkuaD01/scikit-learn#2 or #22269
Labels
Documentation good first issue Easy with clear instructions to resolve module:naive_bayes

Comments

@rmalouf
Copy link
rmalouf commented Mar 7, 2018 •
edited
Loading

A tiny error: PR #9131 prohibits values of alpha too close to zero, but the documentation for BernoulliNB and MultinomialNB still say that alpha=0 is permitted to disable smoothing
@jnothman jnothman added Documentation good first issue Easy with clear instructions to resolve help wanted labels Mar 7, 2018
@bhaskarb
Copy link
bhaskarb commented Mar 8, 2018

Can i make the change for the same? Looks like it says \alpha \ge 0 in the case of MultinomialNB, I do not see a reference to the this in the case of BernoulliNB. I was looking at the naive_bayes.rst file in the source.Thanks in advance

@jnothman
Copy link
Member
jnothman commented Mar 8, 2018 via email

bhaskarb pushed a commit to bhaskarb/scikit-learn that referenced this issue Mar 8, 2018
Documentaion for scikit-learn#10772 MultinomialNB alpha>0
66d6c74
@bhaskarb bhaskarb mentioned this issue Mar 8, 2018
Closed
@divayjindal95
Copy link

Hi, is this done ?

@amueller
Copy link
Member

@divayjindal95 there's an open PR in #10775 as you can see above but it's not entirely clear what the way forward is.

@amueller amueller added Easy Well-defined and straightforward way to resolve help wanted labels Aug 21, 2018
@meghanabhange
Copy link

Hi,
Is this issue still active? Has the course of action been decided on?

@jnothman
Copy link
Member
jnothman commented Oct 9, 2019 via email

@arka204 arka204 mentioned this issue Mar 22, 2020
Closed
@hongshaoyang hongshaoyang mentioned this issue Nov 10, 2020
Closed
@alex-greff alex-greff mentioned this issue Mar 5, 2021
Merged
@cmarmo cmarmo added help wanted module:naive_bayes and removed Easy Well-defined and straightforward way to resolve labels Jan 15, 2022
@Micky774 Micky774 mentioned this issue Jan 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation good first issue Easy with clear instructions to resolve module:naive_bayes
Projects
None yet
Milestone
No milestone
7 participants
@jnothman @amueller @rmalouf @bhaskarb @cmarmo @divayjindal95 @meghanabhange
0