Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the repo’s pinned PyTorch XPU dependency to Torch 2.11 and adjusts CI to install from the PyTorch XPU test wheel index, seemingly to validate compatibility ahead of a stable release.
Changes:
- Switch Torch pin from
2.10.0+xputo2.11.0+xpuin bothrequirements.txtandpyproject.toml. - Change the PyTorch wheel source from the stable XPU index to the
whl/test/xpuindex. - Update the unit test workflow to build/run using a
xpu-kernel-ci-image:test-211Docker tag and install with the test XPU index.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
requirements.txt |
Moves default install to Torch 2.11 XPU and the PyTorch test XPU wheel index. |
pyproject.toml |
Updates build-system Torch requirement to 2.11 XPU. |
.github/workflows/ut.yaml |
Updates CI docker tag to test-211 and switches install to the test XPU wheel index. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| docker build -t xpu-kernel-ci-image:test-211 -f Dockerfile.xpu . | ||
| docker tag xpu-kernel-ci-image:test-211 ${{ env.REGISTRY }}/xpu-kernel-ci-image:test-211 | ||
| docker push ${{ env.REGISTRY }}/xpu-kernel-ci-image:test-211 |
There was a problem hiding this comment.
The workflow no longer tags/pushes xpu-kernel-ci-image:latest (it now only produces :test-211). Downstream workflows (e.g. wheel-per-commit.yaml) still run in localhost:5000/xpu-kernel-ci-image:latest, so they’ll keep using a stale image or fail if :latest doesn’t exist on the runner. Consider continuing to tag/push :latest as well, or update all dependent workflows to use :test-211 consistently.
d3ec13a to
0c29c03
Compare
Signed-off-by: Kunshang Ji <kunshang.ji@intel.com>
bddd55b to
32fd03f
Compare
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS ABOVE HAVE BEEN CONSIDERED.
Purpose
Test Plan
Test Result
(Optional) Documentation Update
BEFORE SUBMITTING, PLEASE READ https://docs.vllm.ai/en/latest/contributing (anything written below this line will be removed by GitHub Actions)