-
Notifications
You must be signed in to change notification settings - Fork 2
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
Adding fastNaiveBayes as engine to naive_bayes() #5
Comments
I'm not a fan of this. It is only fast because it pushes the work on the user. I think that we would end up fielding issues in our repos because it deliberately operates differently than most packages. |
Alright, sounds reasonable. |
what about |
That looks fine. Should I or do you want to take a shot at it? |
I can take a shot at it |
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue. |
Would you be open for a PR adding https://github.com/mskogholt/fastNaiveBayes as an engine to
naive_bayes()
?The text was updated successfully, but these errors were encountered: