Skip to content

[Docs] Adds end-to-end xgboost tutorial #52383

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 78 commits into from
May 27, 2025
Merged

[Docs] Adds end-to-end xgboost tutorial #52383

merged 78 commits into from
May 27, 2025

Conversation

crypdick
Copy link
Contributor

Related issue number

N/A

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@crypdick crypdick requested review from a team as code owners April 16, 2025 23:11
@crypdick crypdick changed the title Adds end-to-end xgboost tutorial [docs] Adds end-to-end xgboost tutorial Apr 16, 2025
@crypdick crypdick added the docs An issue or change related to documentation label Apr 16, 2025
crypdick added 3 commits April 16, 2025 16:15
@crypdick
clean up orphaned mlflow file
99c1f93 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
rm outdated testing readme
5354801 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
make linter happy
e7e53e8 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick crypdick changed the title [docs] Adds end-to-end xgboost tutorial [wip][docs] Adds end-to-end xgboost tutorial Apr 17, 2025
@crypdick
fix name of release test
87852b3 
Signed-off-by: Ricardo Decal <[email protected]>
aslonnie
aslonnie reviewed Apr 21, 2025
View reviewed changes
Copy link
Collaborator

@aslonnie aslonnie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(skipping review for now.. this PR is too large to review in its current shape.)

@crypdick crypdick mentioned this pull request May 7, 2025
Merged
@crypdick
Added manual test for gce
30775e7 
Signed-off-by: Ricardo Decal <[email protected]>
crypdick and others added 17 commits May 22, 2025 14:47
@crypdick
rewrite tests to use pure python instead of pytest to avoid sys path …
ff21d27 
…problems

Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
Revert "add dist_xgboost into py_modules runtime env"
dee181e 
This reverts commit 0322817.
@angelinalg
fix maxdepth in index file
0b629c5 
Signed-off-by: angelinalg <[email protected]>
@angelinalg
Merge branch 'docs/e2e-xgboost' of github.com:ray-project/ray into do…
872902b 
…cs/e2e-xgboost
@crypdick
Revert "Revert "add dist_xgboost into py_modules runtime env""
5524131 
This reverts commit dee181e.
@angelinalg
fix titles and links
f724a71 
Signed-off-by: angelinalg <[email protected]>
@angelinalg
Merge branch 'docs/e2e-xgboost' of github.com:ray-project/ray into do…
f52c2a0 
…cs/e2e-xgboost
@crypdick
switch from /mnt/user_storage to /mnt/cluster_storage
d7184d8 
Signed-off-by: Ricardo Decal <[email protected]>
@angelinalg
Merge branch 'master' into docs/e2e-xgboost
c28b815
@angelinalg
typos
919045b 
Signed-off-by: angelinalg <[email protected]>
@angelinalg
Merge branch 'docs/e2e-xgboost' of github.com:ray-project/ray into do…
b194d67 
…cs/e2e-xgboost
@crypdick
add debug logs for ray storage
dcda17b 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
Merge remote-tracking branch 'origin' into docs/e2e-xgboost
f306412
@crypdick
try adding pythonpath directly
15ce3d1 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
unpatch local storage path
2caa495 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
lint
d8ee523 
Signed-off-by: Ricardo Decal <[email protected]>
@crypdick
Merge branch 'master' into docs/e2e-xgboost
2df44dd
@GokuMohandas
Copy link
Contributor

LGTM

@angelinalg angelinalg merged commit 8b3d977 into master May 27, 2025
5 checks passed
@angelinalg angelinalg deleted the docs/e2e-xgboost branch May 27, 2025 16:27
@crypdick crypdick mentioned this pull request May 27, 2025
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@angelinalg angelinalg angelinalg approved these changes

@aslonnie aslonnie aslonnie approved these changes

@hongpeng-guo hongpeng-guo Awaiting requested review from hongpeng-guo

@justinvyu justinvyu Awaiting requested review from justinvyu

@matthewdeng matthewdeng Awaiting requested review from matthewdeng

@raulchen raulchen Awaiting requested review from raulchen

@woshiyyya woshiyyya Awaiting requested review from woshiyyya

Assignees
No one assigned
Labels
docs An issue or change related to documentation go add ONLY when ready to merge, run all tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@crypdick @GokuMohandas @aslonnie @angelinalg @hainesmichaelc