-
Notifications
You must be signed in to change notification settings - Fork 525
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
[REVIEW] Additional Naive Bayes improvements #1891
[REVIEW] Additional Naive Bayes improvements #1891
Conversation
rerun tests |
…behavior in client.scatter
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.
Pending CI, and with the github issue raised in Dask, lgtm
rerun tests |
rerun tests |
1 similar comment
rerun tests |
rerun tests |
… the logs (not after 50s)
…/cuml into fea-naive_bayes_perf_imp Conflicts: python/cuml/test/dask/test_label_binarizer.py
rerun tests |
rerun tests |
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.
Approving the new changes
This PR contains improvements to Naive Bayes speed and memory usage. This should be merged after dask/distributed#3584.