Upgrade to new vllm commit #3719
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:
If CI fails, you can run linting and testing checks locally according Contributing and Testing. |
wangxiyuan
changed the title
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
wxsIcey
force-pushed
the
fix_main_final
branch
from
315a0e0 to
71af5bf
Compare
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
…lm main Signed-off-by: MengqingCao <cmq0113@163.com> Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: Icey <1790571317@qq.com>
wxsIcey
force-pushed
the
fix_main_final
branch
from
66f53b7 to
f96a942
Compare
MengqingCao
reviewed
Oct 25, 2025
There was a problem hiding this comment.
Thanks for this hard work! I only have 1 question, I think this pr will not work with Qwen3-Next because actually I missed the Qwen3-Next in the previous PR. Could you test Qwen3-Next with #3741 upon this pr?
MengqingCao
reviewed
Oct 25, 2025
| if not model_config.is_multimodal_model and \ | ||
| structured_outputs_config.backend == "auto" and \ | ||
| not getattr(scheduler_config, "scheduler_delay_factor", 0) > 0 and \ | ||
| not scheduler_config.send_delta_data and \ |
There was a problem hiding this comment.
maybe we need to guaranteed compatibility with 0.11.0 instead of remove it directly?
wangxiyuan
approved these changes
Oct 25, 2025
liziyu179
pushed a commit
to nwpu-zxr/vllm-ascend
that referenced
this pull request
Oct 25, 2025
### What this PR does / why we need it? Upgrade to new vllm commit: vllm-project/vllm@c9461e0 - Fix many imports, caused by vllm-project/vllm#26908 - Fix import ```sha256```, caused by vllm-project/vllm#27169 - Remove ```SchedulerConfig.send_delta_data```, caused by vllm-project/vllm#27142 - Fix ```FusedMoE``` because of dual stream execution, caused by vllm-project/vllm#26440 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? CI passed with new added/existing test. - vLLM version: v0.11.0rc3 - vLLM main: vllm-project/vllm@17c540a --------- Signed-off-by: MengqingCao <cmq0113@163.com> Signed-off-by: Icey <1790571317@qq.com> Co-authored-by: MengqingCao <cmq0113@163.com> Signed-off-by: nwpu-zxr <zhouxuerong2@huawei.com>
wangxiyuan
pushed a commit
that referenced
this pull request
Oct 28, 2025
### What this PR does / why we need it? vllm-project/vllm@c9461e0 Fix ```spec decode rejection sampler```, caused by vllm-project/vllm#26060 Fix some ```import```, caused by vllm-project/vllm#27374 Fix ```scheduler_config.send_delta_data```, caused by #3719 Fix ```init_with_cudagraph_sizes```, caused by vllm-project/vllm#26016 Fix ```vl model```of replacing PatchEmbed's conv3d to linear layer, caused by vllm-project/vllm#27418 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? CI passed with new added/existing test. - vLLM version: v0.11.0rc3 - vLLM main: vllm-project/vllm@c9461e0 --------- Signed-off-by: Icey <1790571317@qq.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it?
Upgrade to new vllm commit: vllm-project/vllm@c9461e0
vllm.utilsvllm#26908sha256, caused by [Misc] Move utils to avoid conflicts with stdlib, and move tests vllm#27169SchedulerConfig.send_delta_data, caused by [V0 Deprecation] Remove V0 executors vllm#27142FusedMoEbecause of dual stream execution, caused by [Performance] Dual stream execution of "shared_experts" and "selected_experts" inside FusedMoE vllm#26440Does this PR introduce any user-facing change?
N/A
How was this patch tested?
CI passed with new added/existing test.