Skip to content

docs(tune): fix import count in CIFAR PyTorch tutorial#62756

Open
RudrenduPaul wants to merge 2 commits into
ray-project:masterfrom
RudrenduPaul:docs/fix-cifar-imports-count
Open

docs(tune): fix import count in CIFAR PyTorch tutorial#62756
RudrenduPaul wants to merge 2 commits into
ray-project:masterfrom
RudrenduPaul:docs/fix-cifar-imports-count

Conversation

@RudrenduPaul
Copy link
Copy Markdown

Why are these changes needed?

The CIFAR PyTorch Tune tutorial says:

Only the last three imports are for Ray Tune.

But the actual import cell only has two Ray Tune imports:

from ray import tune
from ray.tune.schedulers import ASHAScheduler

This updates the narrative text to match the actual code.

Related issue

Fixes #61250

Checks

  • I've signed off all commits with git commit -s (N/A — trivial text correction)
  • I've run the relevant tests for my changes
  • I've made sure the tests are passing

The CIFAR PyTorch tutorial imports only 2 Ray Tune items (`from ray
import tune` and `from ray.tune.schedulers import ASHAScheduler`),
not 3 as stated in the narrative text. Corrects the count to match
the actual code.

Fixes ray-project#61250
@RudrenduPaul RudrenduPaul requested a review from a team as a code owner April 19, 2026 05:25
Copy link
Copy Markdown
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 the documentation in the PyTorch CIFAR example notebook to correctly reflect the number of Ray Tune imports. I have no feedback to provide.

@ray-gardener ray-gardener Bot added tune Tune-related issues docs An issue or change related to documentation community-contribution Contributed by the community labels Apr 19, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions Bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label May 3, 2026
Copy link
Copy Markdown
Member

@pseudo-rnd-thoughts pseudo-rnd-thoughts left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, LGTM

@pseudo-rnd-thoughts pseudo-rnd-thoughts added go add ONLY when ready to merge, run all tests and removed stale The issue is stale. It will be closed within 7 days unless there are further conversation labels May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Contributed by the community docs An issue or change related to documentation go add ONLY when ready to merge, run all tests tune Tune-related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Incorrect count of Ray Tune imports in CIFAR PyTorch example

3 participants