Skip to content

Conversation

Potabk
Copy link
Collaborator

@Potabk Potabk commented Apr 23, 2025

What this PR does / why we need it?

Add deepseek-v2-lite test, part of #499

Does this PR introduce any user-facing change?

How was this patch tested?

Potabk added 4 commits April 30, 2025 03:35
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
Signed-off-by: wangli <wangli858794774@gmail.com>
@Potabk
Copy link
Collaborator Author

Potabk commented Apr 30, 2025

@wangxiyuan This is ready

pytest -sv tests/compile
else
pytest -sv tests/multicard/test_offline_inference_distributed.py
pytest -sv -k "QwQ" tests/multicard/test_offline_inference_distributed.py
Copy link
Collaborator

Choose a reason for hiding this comment

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

why we need -k here for multicard test only?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Because I'm not sure where the pytest cache affects the distributed environment, I think this method does not have any additional consumption. The purpose of doing this is to easily clean up the cache of the distributed environment. and Ensures that the test can still run normally one by one

@wangxiyuan wangxiyuan merged commit 8e4e791 into vllm-project:main May 12, 2025
13 checks passed
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