Skip to content

[Data][docs] Update the doc's minimum version to support build_processor#61757

Merged
kouroshHakha merged 1 commit into
ray-project:masterfrom
dancingactor:data_llm_doc
Apr 23, 2026
Merged

[Data][docs] Update the doc's minimum version to support build_processor#61757
kouroshHakha merged 1 commit into
ray-project:masterfrom
dancingactor:data_llm_doc

Conversation

@dancingactor

@dancingactor dancingactor commented Mar 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Update the minimum version in the documentation's installation command to support the build_processor function in the below example in the doc. build_processor was called build_llm_processor before 2.53.0

Change

pip install -U "ray[data, llm]>=2.49.1" -> pip install -U "ray[data, llm]>=2.53.0"

Related issues

Relates to #58484

Additional information

Before the fix

Traceback (most recent call last):
  File "/Users/ryanchen/github/learning/ray/practice.py", line 2, in <module>
    from ray.data.llm import vLLMEngineProcessorConfig, build_processor
ImportError: cannot import name 'build_processor' from 'ray.data.llm' (/Users/ryanchen/github/learning/ray/venv/lib/python3.10/site-packages/ray/data/llm.py)
@dancingactor dancingactor requested a review from a team as a code owner March 16, 2026 06:29
@dancingactor dancingactor changed the title [Data][docs] Update the minimum version to support build_processor Mar 16, 2026
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Mar 16, 2026
View reviewed changes

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

Copy link
Copy Markdown
Contributor

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 minimum required version of ray in the documentation for working with LLMs. The change from >=2.49.1 to >=2.53.0 correctly aligns the installation command with the build_processor function used in the example code, which was introduced in version 2.53.0. This resolves the ImportError described in the pull request. The change is accurate and I have no further comments.
@ray-gardener ray-gardener Bot added docs An issue or change related to documentation data Ray Data-related issues llm community-contribution Contributed by the community labels Mar 16, 2026
@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown

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 Mar 30, 2026
@dancingactor

dancingactor commented Mar 30, 2026

Copy link
Copy Markdown
Contributor Author

@goutamvenkat-anyscale @bveeramani This is a very minor fix. Could you please take a quick look when you have a moment? Thanks!
@dancingactor
[Data][docs] Update the minimum version to support build_processor
61cfae9 
Signed-off-by: dancingactor <s990346@gmail.com>
@bveeramani

bveeramani commented Mar 30, 2026

Copy link
Copy Markdown
Member

Hey @dancingactor , thanks for the PR! I actually don't own this code.

@jeffreyjeffreywang could you take a look?
jeffreywang88
jeffreywang88 approved these changes Mar 30, 2026
View reviewed changes

@jeffreywang88 jeffreywang88 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great find, thanks for the fix!
@github-actions github-actions Bot added unstale A PR that has been marked unstale. It will not get marked stale again if this label is on it. and removed stale The issue is stale. It will be closed within 7 days unless there are further conversation labels Mar 31, 2026
@dancingactor

dancingactor commented Mar 31, 2026

Copy link
Copy Markdown
Contributor Author

@bveeramani Sorry for the wrong tag! Thanks for helping!

@jeffreywang-anyscale Thanks for your review!
@jeffreywang88 jeffreywang88 added the go add ONLY when ready to merge, run all tests label Apr 1, 2026
@dancingactor

dancingactor commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

@jeffreywang-anyscale Could you please help merge this PR, thanks!
@kouroshHakha kouroshHakha merged commit a713478 into ray-project:master Apr 23, 2026
8 checks passed
Lucas61000 pushed a commit to Lucas61000/ray that referenced this pull request May 15, 2026
@dancingactor
[Data][docs] Update the doc's minimum version to support build_proces…
0215828 
…sor (ray-project#61757)

Signed-off-by: dancingactor <s990346@gmail.com>
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 data Ray Data-related issues docs An issue or change related to documentation go add ONLY when ready to merge, run all tests llm unstale A PR that has been marked unstale. It will not get marked stale again if this label is on it.

4 participants

@dancingactor @bveeramani @kouroshHakha @jeffreywang88