-
Notifications
You must be signed in to change notification settings - Fork 596
[Fix] Refactor and fix dist test to e2e full test #3808
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
Conversation
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
👋 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. |
1e3867e to
f59a428
Compare
|
LGTM, plz also take a look @wangxiyuan @Yikun This will trigger test on a3 4 cards per pr |
f59a428 to
04e2c2d
Compare
2b1b9e3 to
2673cff
Compare
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
dc78848 to
71a2c0e
Compare
6bf7c69 to
973d82a
Compare
3fa2a05 to
919a7a7
Compare
Add accuracy test for qwen3-8b-w8a8 - vLLM version: v0.11.0rc3 - vLLM main: vllm-project/vllm@c9461e0 Signed-off-by: hfadzxy <starmoon_zhang@163.com>
919a7a7 to
77bb8f7
Compare
### What this PR does / why we need it? Fix ci test on A3 1. delete lables 2. fix filter yaml file name 3. refactor dist test to e2e full test 4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP & test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of vllm-project#3895 - vLLM version: v0.11.0 - vLLM main: vllm-project/vllm@83f478b Signed-off-by: hfadzxy <starmoon_zhang@163.com>
### What this PR does / why we need it? Fix ci test on A3 1. delete lables 2. fix filter yaml file name 3. refactor dist test to e2e full test 4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP & test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of vllm-project#3895 - vLLM version: v0.11.0 - vLLM main: vllm-project/vllm@83f478b Signed-off-by: hfadzxy <starmoon_zhang@163.com> Signed-off-by: luolun <luolun1995@cmbchina.com>
### What this PR does / why we need it? Fix ci test on A3 1. delete lables 2. fix filter yaml file name 3. refactor dist test to e2e full test 4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP & test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of vllm-project#3895 - vLLM version: v0.11.0 - vLLM main: vllm-project/vllm@83f478b Signed-off-by: hfadzxy <starmoon_zhang@163.com> Signed-off-by: hwhaokun <haokun0405@163.com>
### What this PR does / why we need it? Fix ci test on A3 1. delete lables 2. fix filter yaml file name 3. refactor dist test to e2e full test 4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP & test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of vllm-project#3895 - vLLM version: v0.11.0 - vLLM main: vllm-project/vllm@83f478b Signed-off-by: hfadzxy <starmoon_zhang@163.com> Signed-off-by: nsdie <yeyifan@huawei.com>
What this PR does / why we need it?
Fix ci test on A3
Does this PR introduce any user-facing change?
How was this patch tested?