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

Add initial draft of ML overview document #11114

Merged
merged 4 commits into from May 13, 2024
Merged

Add initial draft of ML overview document #11114

merged 4 commits into from May 13, 2024

Conversation

mrocklin
Copy link
Member

@mrocklin mrocklin commented May 9, 2024

See #11095

cc @jrbourbeau @scharlottej13

This isn't yet done. Some TODOs:

  • Make better example for Optuna
  • Make better example for XGBoost
  • Add more example links throughout

But I figured I'd pause here and ask for large scale feedback

@mrocklin
Copy link
Member Author

mrocklin commented May 9, 2024

I also added a video, but maybe that's a distraction. Mostly I used it because I saw it in the Optuna docs

Copy link
Contributor

github-actions bot commented May 9, 2024

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

     15 files  ± 0       15 suites  ±0   3h 24m 49s ⏱️ + 3m 40s
 13 123 tests + 2   12 191 ✅ + 1     931 💤 ±0  1 ❌ +1 
162 492 runs  +24  142 455 ✅ +14  20 036 💤 +9  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 456d3bd. ± Comparison against base commit aabc53d.

♻️ This comment has been updated with latest results.

@mrocklin
Copy link
Member Author

TODOs are done. I think that this is maybe ok. @scharlottej13 if you're around and can take a look that would be welcome. I'm hopeful that this will help with dask.org links.

docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
docs/source/ml.rst Outdated Show resolved Hide resolved
Co-authored-by: Sarah Charlotte Johnson <scharlottej13@gmail.com>
@scharlottej13
Copy link
Contributor

(could be for a future iteration) I could also see including a joblib + scikit-learn section. Maybe there are also utility functions from dask-ml that people commonly use?

@mrocklin
Copy link
Member Author

(could be for a future iteration) I could also see including a joblib + scikit-learn section. Maybe there are also utility functions from dask-ml that people commonly use?

I don't really see this all that often (xgboost and pytorch seem to be more common with dask workloads). I could be wrong though. I have no objection to people adding stuff here in the future.

For now I'm going to merge this.

@mrocklin mrocklin merged commit f511e93 into dask:main May 13, 2024
26 of 27 checks passed
@mrocklin mrocklin deleted the ml-doc branch May 13, 2024 17:41
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

Successfully merging this pull request may close these issues.

None yet

2 participants