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

[DOC] Update ML docs to reflect recent changes #1022

Closed
sarahyurick opened this issue Jan 31, 2023 · 1 comment · Fixed by #1043
Closed

[DOC] Update ML docs to reflect recent changes #1022

sarahyurick opened this issue Jan 31, 2023 · 1 comment · Fixed by #1043
Labels
documentation Improvements or additions to documentation

Comments

@sarahyurick
Copy link
Collaborator

sarahyurick commented Jan 31, 2023

After the changes in #962, we should update some of our machine learning docs such as https://dask-sql.readthedocs.io/en/latest/machine_learning.html to include backend agnostic ML models. Furthermore, it would be a good idea to revisit other ML changes made recently (#832, #855, #886, #926, #1007, and #976) and consider updating the documentation accordingly.

We still have some debugging to do on the GPU-ML side (see #943 and #1020), so it might be a good idea to make a note of known limitations and/or include working examples once these issues are addressed.

I also think this would be a good chance to close some other existing issues around our ML docs, like #130 and #397.

@sarahyurick sarahyurick added the documentation Improvements or additions to documentation label Jan 31, 2023
@sarahyurick
Copy link
Collaborator Author

On the GPU XGBoost side, it would also be worth emphasizing the gpu_hist parameter.

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

Successfully merging a pull request may close this issue.

1 participant