Skip to content
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

improvements in naive_bayes #108

Closed
fabianp opened this issue Mar 30, 2011 · 4 comments
Closed

improvements in naive_bayes #108

fabianp opened this issue Mar 30, 2011 · 4 comments

Comments

@fabianp
Copy link
Member

fabianp commented Mar 30, 2011

  • across the whole module estimated parameters do not end with underscore
    • non-standard names are used for some variables, like unique_y (called classes in SGD and labels in SVM), we should think about unifying these.
@larsmans
Copy link
Member

I'd go for labels. By the estimated parameters, do you mean proba_y, theta and sigma? Wouldn't changing those break pickle'd GNB instances?

@larsmans
Copy link
Member

Commit d7e64be introduced some changes to Gaussian Naive Bayes, but more work remains to be done. It would be elegant to reformulate GNB as a linear classifier, just like Multinomial NB.

@amueller
Copy link
Member

amueller commented Jan 7, 2012

@fabianp do you think the issue is resolved? Or is there something else to do?

@amueller
Copy link
Member

The unique_y was also fixed at some point, classes_ is now used everywhere as far as I can tell. I hope #890 will help unify things even more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants