Skip to content

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

Merged
angelinalg merged 2 commits into
ray-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 into
ray-project:masterfrom
nrghosh:nrghosh/fix-data-vlm-doc

Conversation

@nrghosh

@nrghosh nrghosh commented Nov 26, 2025

Copy link
Copy Markdown
Contributor

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 Nov 26, 2025
@richardliaw richardliaw self-assigned this Nov 26, 2025

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.

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

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.

comment is specifically on L200

Copy link
Copy Markdown
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 Dec 3, 2025
@nrghosh nrghosh requested a review from a team December 3, 2025 03:15

@angelinalg angelinalg 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.

stamp

@angelinalg angelinalg merged commit 1ab7ee2 into ray-project:master Dec 3, 2025
6 checks passed
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

3 participants