Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[doc][misc] bump up py version in installation doc #6119

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

youkaichao
Copy link
Member

many people (including me) just copy-paste the command from doc. we should recommend the most extensively tested python version (python 3.10, used in ci), so that users' environment is more aligned with our ci, and we can catch more bugs there.

in addition, pipeline parallel currently requires python 3.10, as there are some more work to do to support python 3.8/3.9 in terms of asyncio default event loop.

@andoorve is trying to fix it. orthogonally, we can bump up the recommended python version so that users have better experience with pipeline parallel.

@robertgshaw2-redhat robertgshaw2-redhat enabled auto-merge (squash) July 3, 2024 22:50
@youkaichao youkaichao disabled auto-merge July 3, 2024 22:51
@youkaichao youkaichao merged commit 966fe72 into vllm-project:main Jul 3, 2024
17 of 19 checks passed
@youkaichao youkaichao deleted the bump_doc_py branch July 3, 2024 22:52
robertgshaw2-redhat pushed a commit to neuralmagic/nm-vllm that referenced this pull request Jul 7, 2024
xjpang pushed a commit to xjpang/vllm that referenced this pull request Jul 8, 2024
xjpang pushed a commit to xjpang/vllm that referenced this pull request Jul 24, 2024
Alvant pushed a commit to compressa-ai/vllm that referenced this pull request Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants