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.

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

[Doc] GraphStorm API example notebook 6 converting API examples to run with GraphStorm CLIs #1049

Merged
merged 42 commits into from
Oct 1, 2024
Merged

[Doc] GraphStorm API example notebook 6 converting API examples to run with GraphStorm CLIs #1049

merged 42 commits into from
Oct 1, 2024

Conversation

zhjwy9343
Copy link
Contributor

Issue #, if available:

Description of changes:
This PR is the 3rd and final PR of example Notebook 6, which shows how to convert pipelines built with GraphStorm APIs to Python scripts that can run with GraphStorm CLIs.

The preview page:

  1. Notebook overview page: https://james4graphstorm.readthedocs.io/en/james_nb6_api2cli/api/notebooks/index.html
  2. Notebook 6 page: https://james4graphstorm.readthedocs.io/en/james_nb6_api2cli/api/notebooks/Notebook_6_API2CLI.html

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@zhjwy9343 zhjwy9343 added documentation Improvements or additions to documentation ready able to trigger the CI 0.4 labels Sep 27, 2024
classicsong
classicsong reviewed Sep 28, 2024
View reviewed changes
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
docs/source/api/notebooks/demo_run_train.py Show resolved Hide resolved
docs/source/api/notebooks/index.rst Outdated Show resolved Hide resolved
docs/source/cli/model-training-inference/single-machine-training-inference.rst Show resolved Hide resolved
@thvasilo thvasilo self-requested a review September 30, 2024 16:11
@zhjwy9343 zhjwy9343 removed the ready able to trigger the CI label Sep 30, 2024
zhjwy9343 and others added 8 commits September 30, 2024 13:20
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
2ab8f7f 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
331592d 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
d01e6c6 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
87ea784 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
1945e82 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
dff6f6f 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
2c55475 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
c4f1ad6 
Co-authored-by: xiang song(charlie.song) <[email protected]>
zhjwy9343 and others added 20 commits September 30, 2024 13:30
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
f0bc75a 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
b773d05 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
2cbfa32 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
cf2e3b7 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
e1d6d32 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
073f1df 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
455722b 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
e27e62b 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
14e2c86 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
395cf79 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
27a71d4 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
7fe7be9 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
1f31fbc 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
54fd455 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
d9cdd96 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
e9e5e43 
Co-authored-by: xiang song(charlie.song) <[email protected]>
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/Notebook_6_API2CLI.ipynb
9c45912 
Co-authored-by: xiang song(charlie.song) <[email protected]>
update notebook 6
ea20517
@zhjwy9343 @classicsong
Update docs/source/api/notebooks/index.rst
e7fefb9 
Co-authored-by: xiang song(charlie.song) <[email protected]>
update index.rst
6601b55
@zhjwy9343 zhjwy9343 added the ready able to trigger the CI label Sep 30, 2024
classicsong
classicsong approved these changes Oct 1, 2024
View reviewed changes
Copy link
Contributor

@classicsong classicsong left a comment

Choose a reason for hiding this comment

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

Minor comments.
LGTM.

docs/source/api/notebooks/Notebook_6_API2CLI.ipynb Outdated Show resolved Hide resolved
@zhjwy9343 zhjwy9343 merged commit e21e040 into awslabs:main Oct 1, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@classicsong classicsong classicsong approved these changes

@thvasilo thvasilo Awaiting requested review from thvasilo

Assignees

@zhjwy9343 zhjwy9343

Labels
0.4 documentation Improvements or additions to documentation ready able to trigger the CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhjwy9343 @classicsong @thvasilo