Adding "Basic Models Test" and "Multi-Modal Models Test (Extended) 3" in AMD Pipeline#18106
Conversation
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
|
👋 Hi! Thank you for contributing to the vLLM project. 💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels. Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging. To run CI, PR reviewers can either: Add 🚀 |
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com> Co-authored-by: Cyrus Leung <cyrus.tl.leung@gmail.com>
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com> Co-authored-by: Cyrus Leung <cyrus.tl.leung@gmail.com>
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
|
Pre-commit is still failing |
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
…com:Alexei-V-Ivanov-AMD/vllm into amd_experimental_all_tests_updated_20250512
Both tests are green (https://buildkite.com/vllm/fastcheck/builds/23953) Please merge this PR. Thanks! |
|
|
||
|
|
||
| @pytest.mark.skipif(current_platform.is_rocm(), | ||
| reason="Llama-3.2-1B-Instruct, Ilama-3.2-1B produce memory access fault.") |
There was a problem hiding this comment.
You need to fix pre-commit before I can merge the PR
|
can you fix precommit? |
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com>
Fixed. |
… in AMD Pipeline (vllm-project#18106) Signed-off-by: Alexei V. Ivanov <alexei.ivanov@amd.com> Co-authored-by: Cyrus Leung <cyrus.tl.leung@gmail.com> Signed-off-by: Yuqi Zhang <yuqizhang@google.com>
Adding "Basic Models Test" and "Multi-Modal Models Test (Extended) 3" in AMD Pipeline
Signed-off-by: Alexei V. Ivanov alexei.ivanov@amd.com