Skip to content

[data][llm] Expose vision_preprocess and vision_postprocess in VLM docs#59012

Merged
angelinalg merged 2 commits intoray-project:masterfrom
nrghosh:nrghosh/fix-data-vlm-doc
Dec 3, 2025
Merged

[data][llm] Expose vision_preprocess and vision_postprocess in VLM docs#59012
angelinalg merged 2 commits intoray-project:masterfrom
nrghosh:nrghosh/fix-data-vlm-doc

Conversation

@nrghosh
Copy link
Contributor

@nrghosh nrghosh commented Nov 26, 2025

Previous refactors caused the pre/post processing details to be omitted from the working with llms guide

image

Signed-off-by: Nikhil Ghosh <nikhil@anyscale.com>
@nrghosh nrghosh requested review from a team November 26, 2025 20:54
@nrghosh nrghosh marked this pull request as ready for review November 26, 2025 23:48
@richardliaw richardliaw changed the title Expose vision_preprocess and vision_postprocess in VLM docs [data/llm] Expose vision_preprocess and vision_postprocess in VLM docs Nov 26, 2025
@richardliaw richardliaw self-assigned this Nov 26, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we include this None None thing as part of the example snippet?

Copy link
Contributor

Choose a reason for hiding this comment

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

comment is specifically on L200

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sorry missed this - yeah that's not super helpful from user/reader POV, just error handling. removed it.

@richardliaw richardliaw added docs An issue or change related to documentation data Ray Data-related issues llm go add ONLY when ready to merge, run all tests labels Nov 26, 2025
Signed-off-by: Nikhil Ghosh <nikhil@anyscale.com>
@nrghosh nrghosh changed the title [data/llm] Expose vision_preprocess and vision_postprocess in VLM docs [data][llm] Expose vision_preprocess and vision_postprocess in VLM docs Dec 3, 2025
@nrghosh nrghosh requested a review from a team December 3, 2025 03:15
Copy link
Contributor

@angelinalg angelinalg left a comment

Choose a reason for hiding this comment

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

stamp

@angelinalg angelinalg merged commit 1ab7ee2 into ray-project:master Dec 3, 2025
6 checks passed
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Ray Data-related issues docs An issue or change related to documentation go add ONLY when ready to merge, run all tests llm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants