Skip to content

[train][doc] Add get_all_reported_checkpoints and ReportedCheckpoint to API docs#56174

Merged
justinvyu merged 6 commits intoray-project:masterfrom
TimothySeah:tseah/document-get-all-reported
Sep 3, 2025
Merged

[train][doc] Add get_all_reported_checkpoints and ReportedCheckpoint to API docs#56174
justinvyu merged 6 commits intoray-project:masterfrom
TimothySeah:tseah/document-get-all-reported

Conversation

@TimothySeah
Copy link
Contributor

No description provided.

…to API docs

Signed-off-by: Timothy Seah <tseah@anyscale.com>
@TimothySeah TimothySeah requested review from a team as code owners September 2, 2025 21:48
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds get_all_reported_checkpoints and ReportedCheckpoint to the Ray Train API documentation. The new entries are added to the correct sections and maintain the alphabetical order of the existing API lists. The changes look good.

Signed-off-by: Timothy Seah <tseah@anyscale.com>
@ray-gardener ray-gardener bot added docs An issue or change related to documentation train Ray Train Related Issue labels Sep 3, 2025
@TimothySeah TimothySeah added the go add ONLY when ready to merge, run all tests label Sep 3, 2025
:nosignatures:
:toctree: doc/

~train.get_all_reported_checkpoints
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I think you might need to update the return type to fix the formatting.

Image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed so the entire returns docstring is on one line.

This is what I see before my new commit - the List[ReportedCheckpoint] at the top works for me.

Screenshot 2025-09-02 at 8 15 01 PM

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: Timothy Seah <tseah@anyscale.com>
@justinvyu justinvyu merged commit 1d76e55 into ray-project:master Sep 3, 2025
5 checks passed
sampan-s-nayak pushed a commit to sampan-s-nayak/ray that referenced this pull request Sep 8, 2025
…to API docs (ray-project#56174)

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: sampan <sampan@anyscale.com>
jugalshah291 pushed a commit to jugalshah291/ray_fork that referenced this pull request Sep 11, 2025
…to API docs (ray-project#56174)

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: jugalshah291 <shah.jugal291@gmail.com>
wyhong3103 pushed a commit to wyhong3103/ray that referenced this pull request Sep 12, 2025
…to API docs (ray-project#56174)

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: yenhong.wong <yenhong.wong@grabtaxi.com>
dstrodtman pushed a commit that referenced this pull request Oct 6, 2025
…to API docs (#56174)

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Signed-off-by: Douglas Strodtman <douglas@anyscale.com>
landscapepainter pushed a commit to landscapepainter/ray that referenced this pull request Nov 17, 2025
…to API docs (ray-project#56174)

Signed-off-by: Timothy Seah <tseah@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs An issue or change related to documentation go add ONLY when ready to merge, run all tests train Ray Train Related Issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants