Skip to content

[Dependency] Remove stale cuDNN frontend upper bound#42599

Open
mmangkad wants to merge 2 commits into
vllm-project:mainfrom
mmangkad:remove-cudnn-frontend-upper-bound
Open

[Dependency] Remove stale cuDNN frontend upper bound#42599
mmangkad wants to merge 2 commits into
vllm-project:mainfrom
mmangkad:remove-cudnn-frontend-upper-bound

Conversation

@mmangkad
Copy link
Copy Markdown
Contributor

Summary

Remove the stale upper bound on nvidia-cudnn-frontend since the old 1.19.0 upstream issue has been fixed for a while now, matching FlashInfer's current >=1.13.0 requirement and letting vLLM pick up newer cuDNN frontend fixes and improvements

Signed-off-by: Mohammad Miadh Angkad <176301910+mmangkad@users.noreply.github.com>
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

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 requirements/cuda.txt file by removing the upper version constraint on nvidia-cudnn-frontend, which was previously capped at version 1.19.0. I have no feedback to provide.

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant