Skip to content

Conversation

wangxiyuan
Copy link
Collaborator

@wangxiyuan wangxiyuan commented Mar 25, 2025

  1. Make sure the version is string before parse in collect_env
  2. Add basic V1 engine test

@wangxiyuan wangxiyuan force-pushed the fix_collect_env branch 3 times, most recently from 52e56a9 to b1f58dd Compare March 27, 2025 06:30
@wangxiyuan
Copy link
Collaborator Author

wangxiyuan commented Mar 27, 2025

CI is blocked. need this fix be merged first. vllm-project/vllm#15589

@wangxiyuan wangxiyuan changed the title Fix collect env bug Enable V1 engine test Mar 27, 2025
@wangxiyuan wangxiyuan force-pushed the fix_collect_env branch 2 times, most recently from 2ea048f to d16b4fa Compare March 27, 2025 09:09
@wangxiyuan wangxiyuan force-pushed the fix_collect_env branch 5 times, most recently from 1e0c994 to 98467b3 Compare March 28, 2025 01:20
@wangxiyuan wangxiyuan force-pushed the fix_collect_env branch 3 times, most recently from e17827b to cc4f6cd Compare March 28, 2025 10:01
Signed-off-by: wangxiyuan <[email protected]>
e_map)
# TODO: The native params are: atol=2e-2, rtol=0, maybe related to the nan problem
torch.testing.assert_close(output, torch_output, atol=4e-2, rtol=1)
torch.npu.empty_cache()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a little bit wired, pls move this teardown in followup PR

@Yikun Yikun changed the title Enable V1 engine test [Core] Make V1 work and enable V1 engine test Mar 28, 2025
@Yikun Yikun merged commit 31f29b9 into vllm-project:main Mar 28, 2025
18 of 20 checks passed
@wangxiyuan wangxiyuan deleted the fix_collect_env branch March 29, 2025 06:11
ttanzhiqiang pushed a commit to ttanzhiqiang/vllm-ascend that referenced this pull request Apr 27, 2025
1. Make sure the version is string before parse in collect_env
2. Add basic V1 engine test

Signed-off-by: wangxiyuan <[email protected]>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
1. Make sure the version is string before parse in collect_env
2. Add basic V1 engine test

Signed-off-by: wangxiyuan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants