-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Revert recent changes #3845
base: main
Are you sure you want to change the base?
Revert recent changes #3845
Conversation
docs/start/install.md
Outdated
|
||
```bash | ||
pip install --upgrade pip | ||
pip install "sglang[all]>=0.4.3.post2" --find-links https://flashinfer.ai/whl/cu124/torch2.5/flashinfer-python | ||
pip install uv | ||
uv pip install sgl-kernel --force-reinstall --no-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you need this additional command to install sgl kernel? Can you just remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see below.
docs/start/install.md
Outdated
@@ -29,6 +32,7 @@ git clone -b v0.4.3.post2 https://github.com/sgl-project/sglang.git | |||
cd sglang | |||
|
|||
pip install --upgrade pip | |||
pip install sgl-kernel --force-reinstall --no-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me take a look later into this. i will try if it works without installing the kernel.
so far we always included that into the installment docs so i assumed it is needed.
We will fix the install on our side:
|
we don't need to install kernel indeed. i removed it from the docs. |
Motivation
See this issue
@zhaochenyang20