Skip to content

Conversation

@matthewdeng
Copy link
Contributor

Description

Use tune.report instead of train.report.

@matthewdeng matthewdeng requested a review from a team October 22, 2025 16:55
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 updates a jobs test to use tune.report from the ray.tune module instead of train.report. This involves removing the train import and updating the function call. This change improves code clarity and API consistency within the context of a Ray Tune job. The implementation is correct and enhances maintainability. The change is a good step towards a cleaner API usage.

@matthewdeng matthewdeng enabled auto-merge (squash) October 22, 2025 18:28
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Oct 22, 2025
@aslonnie aslonnie disabled auto-merge October 22, 2025 18:40
@aslonnie aslonnie merged commit d027e90 into ray-project:master Oct 22, 2025
6 of 7 checks passed
aslonnie added a commit that referenced this pull request Oct 22, 2025
cherrypick #57995

Signed-off-by: Lonnie Liu <[email protected]>
aslonnie added a commit that referenced this pull request Oct 22, 2025
landscapepainter pushed a commit to landscapepainter/ray that referenced this pull request Nov 17, 2025
## Description
Use `tune.report` instead of `train.report`.

Signed-off-by: Matthew Deng <[email protected]>
Aydin-ab pushed a commit to Aydin-ab/ray-aydin that referenced this pull request Nov 19, 2025
## Description
Use `tune.report` instead of `train.report`.

Signed-off-by: Matthew Deng <[email protected]>
Signed-off-by: Aydin Abiar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants