Skip to content

docs(python): improve Python SDK documentation#931

Merged
SYaoJun merged 3 commits into
apache:mainfrom
Iskander14yo:py-docs
Jun 9, 2026
Merged

docs(python): improve Python SDK documentation#931
SYaoJun merged 3 commits into
apache:mainfrom
Iskander14yo:py-docs

Conversation

@Iskander14yo

Copy link
Copy Markdown
Contributor

Reason for this PR

Improve the Python SDK documentation for first-time users.

Particularly, this updates the Python README and CLI README to clarify installation, quick-start usage, examples, test data setup, CLI commands, and import config behavior.

What changes are included in this PR?

  • Clarify Python SDK install paths
  • Add links to related modules and folders.
  • Document how to download and configure GAR_TEST_DATA for examples and tests.
  • Clean up and correct inconsistencies. I comment here some of them where unsure.

Are these changes tested?

No.

Are there any user-facing changes?

Both Python's READMEs are updated.

Checklist

  • I have performed a self-review of my own code.
  • I have formatted my own code using make cpplint before submitting when changed files are in the cpp directory.
  • I have performed pre-commit run before commit the changed files.
  • I have added tests to prove my changes are effective.

Comment thread python/src/cli/README.md

Wish you a happy use! No newline at end of file
| --------------------------------- | -------------------- |
| `graphar.vertex_chunk_size` | `100` |

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

From this link vertex and edge chunk sizes are recommended to be 2^18 and 2^22 correspondingly. Should we correct values here?

Comment thread python/src/cli/README.md
## Import config file
## Usage

The config file supports `yaml` data type. We provide two reference templates for it: full and mini.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Can you please point me to these templates? I think it's better to have links to them (if this info is not outdated) or delete this block otherwise (what I did).

@SemyonSinchenko SemyonSinchenko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

~LGTM

@SemyonSinchenko

SemyonSinchenko commented Jun 7, 2026

Copy link
Copy Markdown
Member

@SYaoJun @yecol Hi! This is docs update based on user experience and from users :)
I have nothing against merging it. What do you think?

LGTM~

@SemyonSinchenko SemyonSinchenko added the Component:Documentation Improvements or additions to documentation label Jun 7, 2026
@SYaoJun SYaoJun merged commit da85682 into apache:main Jun 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component:Documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants