Skip to content

Add Cosmos3-Edge to the TensorRT-LLM Reasoner cookbook - #347

Open
ConstBob wants to merge 1 commit into
NVIDIA:mainfrom
ConstBob:docs/cosmos3-edge-reasoner-trtllm-main
Open

ConstBob wants to merge 1 commit into
NVIDIA:mainfrom
ConstBob:docs/cosmos3-edge-reasoner-trtllm-main

Conversation

@ConstBob

Copy link
Copy Markdown
Contributor

Summary

Testing

Replay the reviewed Edge Reasoner cookbook onto main after the generate-media skill migration. Nano and Super stay on release:1.3.0rc22. Edge is served from TensorRT-LLM main at or after 39a44fab.

Signed-off-by: Yiming Zhao <yimingz@nvidia.com>
@yaoxu-crypto yaoxu-crypto added the Skill New skill or updating existing skill label Oct 30, 2026
[`39a44fab`](https://github.com/NVIDIA/TensorRT-LLM/commit/39a44fab74f4cfc0d4d498be385e746fdd182872)
(the merge of [PR 20307](https://github.com/NVIDIA/TensorRT-LLM/pull/20307)),
following the
[TensorRT-LLM Generator source-build instructions](#tensorrt-llm-generator),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could we please make the linked source-build steps select a revision that actually includes Edge Reasoner support? This paragraph correctly requires 39a44fab or newer, but the linked TensorRT-LLM Generator instructions still run git checkout cc63e66daf52644f1441e117bb7ba67d20566016 (README line 202). I checked the upstream commit ancestry: cc63e66 is an ancestor of 39a44fab and predates the Edge Reasoner merge, so following that build block literally does not meet this prerequisite. An explicit Edge-specific checkout override at 39a44fab or newer, or corrected linked instructions, would avoid that contradiction while keeping the Nano/Super release-image path unchanged. Thank you!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skill New skill or updating existing skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants