Skip to content

nano_nemotron_vl: fix tensor device mismatch exception when video profiling#39029

Merged
robertgshaw2-redhat merged 3 commits into
vllm-project:mainfrom
netanel-haber:bugfix/nano-nemotron-vl-fix-video-profiling
Apr 5, 2026
Merged

nano_nemotron_vl: fix tensor device mismatch exception when video profiling#39029
robertgshaw2-redhat merged 3 commits into
vllm-project:mainfrom
netanel-haber:bugfix/nano-nemotron-vl-fix-video-profiling

Conversation

@netanel-haber
Copy link
Copy Markdown
Contributor

No description provided.

…filing

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
@netanel-haber netanel-haber requested a review from tomeras91 as a code owner April 5, 2026 14:15
Copy link
Copy Markdown
Contributor

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

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 _create_final_video_embeddings method in vllm/model_executor/models/nano_nemotron_vl.py to ensure that newly created tensors are placed on the same device as the video embeddings, preventing potential device mismatch issues. I have no feedback to provide as there are no review comments.

@robertgshaw2-redhat robertgshaw2-redhat added the ready ONLY add when PR is ready to merge/full CI is needed label Apr 5, 2026
@robertgshaw2-redhat robertgshaw2-redhat enabled auto-merge (squash) April 5, 2026 14:32
@robertgshaw2-redhat robertgshaw2-redhat merged commit d56e952 into vllm-project:main Apr 5, 2026
57 checks passed
HenryTangDev pushed a commit to HenryTangMain/vllm that referenced this pull request Apr 6, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
puririshi98 pushed a commit to puririshi98/vllm that referenced this pull request Apr 7, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
Signed-off-by: Rishi Puri <riship@nvidia.com>
mtparet pushed a commit to blackfuel-ai/vllm that referenced this pull request Apr 9, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
mystous pushed a commit to mystous/vllm_hybrid that referenced this pull request May 10, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
jhu960213 pushed a commit to jhu960213/vllm that referenced this pull request May 20, 2026
…filing (vllm-project#39029)

Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants