Repository navigation
Conversation
|
Can we add some accuracy benchmark instruction or some references of whether this is correct? |
|
Requesting reviews from @nvluxiaoz @qmiao-hub @rickzw so that we can merge this PR. Thanks! |
nvluxiaoz
left a comment
There was a problem hiding this comment.
LGTM as long as ever has validated.
| ": \"${OUTPUT_JSON:?run the setup cell first}\"\n", | ||
| "\n", | ||
| "cd \"$EDGELLM_ROOT\"\n", | ||
| "./build/experimental_models/cosmos3/examples/cosmos3_policy_inference \\\n", |
There was a problem hiding this comment.
I think we need to add --viewPoint concat_view here to match the concatenated DROID camera input. @nvluxiaoz to confirm.
Align the TensorRT-Edge-LLM policy notebook with the vLLM-Omni/SGLang observation (640x540 multiview + pick-and-place prompt) instead of the reasoner still and upstream CLI placeholder.
The v0.11.0 policy CLI defaults to ego_view, so the DROID concat input has to pass --viewPoint concat_view.
38e8d06 to
57f4f47
Compare
The overview was saying every backend covers forward dynamics, inverse dynamics, and policy, while TensorRT-Edge-LLM only runs the Edge policy task.
|
Hi @ever-wong , thank you for the comments. Rebased onto current main. The notebooks now pin TensorRT-Edge-LLM v0.11.0, and the policy command passes I built the v0.11.0 Could you confirm whether this cookbook should stay on the VLA-guide export path until that contract matches, or switch to the experimental builder? I have not re-exported or rebuilt engines on v0.11.0 yet. |
Summary
cookbooks/cosmos3/README.md. Policy needs-DBUILD_EXPERIMENTAL_MODELS=ON.Testing
Tested on an x86 developer GPU (RTX PRO 6000, SM120) with TensorRT-Edge-LLM v0.10.0. Both added notebooks in this PR were run top-to-bottom (
run_with_trt_edge_llm.ipynbandrun_policy_with_trt_edge_llm.ipynb).--task reasoning→llm_build+visual_build→llm_inferenceonrobot_153.jpg. Pipeline finished.--task policy→cosmos3_policy_build→cosmos3_policy_inference.[1,16,10],finite=true, ~195 ms (runtime smoke).On this SM120 box the pipeline succeeded for both notebooks (export, engine build, and C++ inference finished; policy returned
[1,16,10]withfinite=true). Reasoner captions were not usable (text did not matchrobot_153.jpg); policy was only checked as a finite action tensor, not DROID quality.Likely cause: TensorRT engines and ViT/FMHA kernels are SM-specific. The device used for this testing is RTX PRO 6000. Official Edge-LLM targets are Thor / Spark / Orin, etc. I do not have an Official Edge-LLM device (Jetson Thor / DRIVE Thor / DGX Spark / Orin), so caption or policy quality on those SKUs are not checked. x86 is Developer-only; on this SM120 box reasoner captions were not usable and should not be treated as gold. Re-run is needed on Thor, or Spark) before treating quality as validated.
The above issue has been resolved (please refer to the comment threads below).