Skip to content

Skill to check for prebuilt wheels during installation - #48925

Open
ariG23498 wants to merge 4 commits into
vllm-project:mainfrom
ariG23498:installation
Open

ariG23498 wants to merge 4 commits into
vllm-project:mainfrom
ariG23498:installation

Conversation

@ariG23498

Copy link
Copy Markdown
Contributor

While using the VLLM_USE_PRECOMPILED=1 flag during installation with a latest commit (mostly editable installs) this skill would check for the prebuilt wheels in the registry. This is done in order to improve the developer experience of the users.

CC: @hmellor

@claude claude Bot left a comment

Copy link
Copy Markdown

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.

@hmellor hmellor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd make the following tweaks:

  • Don't tell it what cuda versions there are, make it check the machine it's running on
  • Only check before install if the main commit is <2h old so we don't make every agent in the world DDoS the wheels server

@ariG23498

Copy link
Copy Markdown
Contributor Author

@hmellor I have updated the markdown.

@mgoin mgoin mentioned this pull request Oct 1, 2026
6 tasks done
@mergify mergify Bot added the agents Related to information for agents label Oct 1, 2026

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

agents Related to information for agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants