-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
[V1] Feedback Thread #12568
Comments
👍 I have not done a proper benchmark but V1 feels superior, i.e. higher throughput + lower latency, TTFT. I have encountered a possible higher memory consumption issue, but am overall very pleased with the vllm community's hard work on V1. |
Does anyone know about this bug with n>1? Thanks |
Logging is in progress. Current main has a lot more and we will maintain compatibility with V0. Thanks! |
Quick feedback [VLLM_USE_V1=1]:
|
Thanks, both are in progress |
are logprobs output (and specifically prompt logprobs with echo=True) expected to be working with current V1 (0.7.0)? |
Maybe there is a better place to discuss this but the implementation for models that use more than one extra modality is quite non-intuitive. |
Still in progress |
Thanks for fixing metrics logs in 0.7.1! |
I'm either going insane, but with V1 qwen 8b instruct LLM just breaks in fp8 and around 25% of generations are just gibberish, with same running code and everything. Do I need to make a bug report, or it's an expected behaviour and I need some specific setup of sampling params for it to work in v1? |
The V1 engine doesn't seem to support logits processors or min-p filtering. Issue #12678 |
Something is weird with memory calculation in V1 and tensor parallel. Here are 2 cases that I tested recently: vllm 0.7.0 on 2x A6000: Starting normally a 32b-awq model and using Everything works as previously, GPUs both get to ~44-46GB usage Using GPUs both load up to ~24-25GB and it slowly goes up as inference runs. I've seen it go up to 32GB on each GPU. Updating to vllm 0.7.1 and running a 7b-awq model this time, I also noticed that running the above command "normally" the logs show Maximum concurrency at 44x Using V1 I get:
And finally, with vllm 0.7.0 and 4x L4 loading a 32b-awq model with tp 4 works in "normal mode", but OOMs with V1. |
I did a little experiment with DeepSeek-R1 on 8xH200 GPU. vLLM 0.7.0 showed the following results with
In general, vLLM without VLLM_USE_V1 looked more productive. I also tried V0 with
Throughput was still 2 times lower than SGLang in the same benchmark. Today I updated vLLM to the new version (0.7.1) and decided to repeat the experiment. And the results in version V0 have become much better!
But running vLLM with
|
v1 not support T4,are you support? |
Hi @bao231, V1 does not support T4 or older-generation GPUs since the kernel libraries used in V1 (e.g., flash-attn) do not support them. |
V1 support other attention libs?has you plan? @WoosukKwon |
Thanks!
|
Can you provide a more detailed reproduction instruction? cc @WoosukKwon |
Thanks. We are actively working on PP |
Check out #sig-multi-modality in our slack! This is the best place for a discussion like this |
Its pretty hard to follow what you are seeing. Please attach:
Thanks! |
Hi, please see Launch command
|
Please also try same seed, or zero temperature, without speculative decoding to see if problem exists. |
It's related to this pr |
Could you try main again and see if it is fixed? Thanks! |
This comment has been minimized.
This comment has been minimized.
Hi, are you already working on resolving the size mismatch issue when loading the MixtralForCausalLM GGUF model? |
#14915 hi, i meet this bug |
#15046 When trying one of the listed supported models with architecture |
Is it possible to close prefix-caching in V1 ? |
Hi. V1 only supports using the XGrammar backend for structured generation, but XGrammar does not support as many JSON schemas as Outlines. Specifically, I'm using |
You can set |
We are aware and are close to finishing other structured generation backends in V1. Ideally EOW |
I would suggest using |
Have to disable v1 engine due to this small restriction: #15252. Will x-post in the Slack for vis. |
I have documented the results of my experiments comparing the throughput of V0 and V1 in a newly created issue. The findings suggest that when GPU memory is fully utilized, preemption occurs, and V1 fails to demonstrate a significant throughput advantage over V0. Can anyone explain why this happens? |
We've encountered a critical memory leak when using the V1 engine for image inference — system RAM usage exceeds 200 GB over time. Full bug report with reproduction steps and details can be found here: #15294. |
Just wanted to leave a quick comment here that I think the default value of |
Thanks. We have resolved this and will do a hotfix. |
Thanks! |
Can you share more about what is causing OOM? Is this during profiling? The value of |
@robertgshaw2-redhat I'm not sure whether this is during the profiling step but essentially our deployments started running into these errors during engine start-up for the
We do not see the error if we manually set the |
Is there a temporary way to get around the I'm running into the same issue as (#14992) with v0.7.2. |
OOM issue after upgrade to v0.8. Same configuration should work on preceding vllm version. I'm deploying a 72b AWQ model on 8x4090, which works fine with 128k context length. Things go wrong after I upgrade to latest version (0.8.2), no matter what number of I event set |
when l use 8*H20 * 2 to run DeepSeek-R1 with vllm0.8.2, l get a terrible error,please help kids 2025-03-28 07:55:36,878 INFO worker.py:1660 -- Connecting to existing Ray cluster at address: 7.216.55.218:6379... |
@win9killhuaxiong could this be a host OOM? Can you monitor your host memory when this error is thrown? |
not OOM,when l start service,the memory is not change, maybe is ray bug |
Model: RefalMachine/RuadaptQwen2.5-1.5B-instruct I measured model Qwen2.5-1.5B-instruct. I was extremely surprised by the results. Now I see the point of using v1 only for structured output in full accuracy. In other cases, the results will be worse, or about the same. |
V1 seems consistently a lot slower with speculative decode compared to the old engine. Using Qwen 14B+1.5B. |
Hey team!
Would this be something on the immediate roadmap? Thanks for your hard work, V1, in general, looks so promising and more hackable than the slightly over-bloated V0! Edit: I've just learned that the PR is already there: #16188 |
Please leave comments here about your usage of V1, does it work? does it not work? which feature do you need in order to adopt it? any bugs?
For bug report, please file it separately and link the issue here.
For in depth discussion, please feel free to join #sig-v1 in the vLLM Slack workspace.
The text was updated successfully, but these errors were encountered: