Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: xiang song(charlie.song) <classicxsong@gmail.com>
  • Loading branch information
thvasilo and classicsong authored Nov 6, 2024
1 parent a0d7e2b commit 66729f4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/source/advanced/using-graphbolt.rst
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ the partitioned DGL graph files have been created on S3.
After running your distributed partition SageMaker job as normal using
``sagemaker/launch_partition.py``, you next need to launch the
``sagemaker/launch_graphbolt_convert.py`` script, passing as input
the S3 URI where the DistDGL partition data is stored by ``launch_partition.py``,
the S3 URI, where the DistDGL partition data is stored by ``launch_partition.py``,
**plus the suffix `dist_graph`** as that's where GSPartition creates the partition files.

For example, if you used ``--output-data-s3 s3://my-bucket/my-part-graph`` for
Expand Down
2 changes: 1 addition & 1 deletion sagemaker/launch/common_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ def parse_unknown_gs_args(unknown_args: List[str]) -> Dict[str, str]:
The input is a list of arguments, the second element of the tuple
returned by ``argparse.ArgumentParser.parse_known_args()``, which
can be a single string depending on how the arguments were passed.
can be a single string depending on how the arguments are passed.
We must handle cases like
``--target-etype query,clicks,asin query,search,asin``
Expand Down

0 comments on commit 66729f4

Please # to comment.