Skip to content

update to latest Vulkan SDK #7258

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

Merged
merged 1 commit into from
Dec 10, 2024
Merged

update to latest Vulkan SDK #7258

merged 1 commit into from
Dec 10, 2024

Conversation

wdvr
Copy link
Contributor

@wdvr wdvr commented Dec 9, 2024

Summary

In pytorch/pytorch tests, pytorch-linux-jammy-py3-clang12-executorch started to fail today due to a 404 on the Vulkan SDK we use/download (1.2.198.1, 3 years old, URL: https://sdk.lunarg.com/sdk/download/1.2.198.1/linux/vulkansdk-linux-x86_64-1.2.198.1.tar.gz )

Test plan

Looking for some advice how this should be tested

@pytorch-bot pytorch-bot bot added the module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/ label Dec 9, 2024
Copy link

pytorch-bot bot commented Dec 9, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/7258

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit bc9214e with merge base 29f5cac (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 9, 2024
@wdvr
Copy link
Contributor Author

wdvr commented Dec 9, 2024

@pytorchbot label "topic: not user facing"

@wdvr wdvr marked this pull request as ready for review December 9, 2024 23:56
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Dec 10, 2024
pytorch-linux-jammy-py3-clang12-executorch [started to fail](https://github.com/pytorch/pytorch/actions/runs/12244909721/job/34157668780) today due to a 404 on the Vulkan SDK we use/download (1.2.198.1, 3 years old, URL: https://sdk.lunarg.com/sdk/download/1.2.198.1/linux/vulkansdk-linux-x86_64-1.2.198.1.tar.gz )

The Vulkan SDK is probably no longer needed for building Executorch, and is not used down the line for testing.

This PR tests removing the installation of the SDK

pytorch/executorch#7258
Pull Request resolved: #142424
Approved by: https://github.com/huydhn
@SS-JIA
Copy link
Contributor

SS-JIA commented Dec 10, 2024

LGTM, thanks for the fix! The change should be tested automatically by the CI. The pull / unittest / linux / linux-job (pull_request) job will build the Vulkan backend and run a unit test. Since there are no failures, I believe this change is ok.

@wdvr wdvr merged commit 61bd2b8 into main Dec 10, 2024
74 of 76 checks passed
@wdvr wdvr deleted the wdvr/vulkan_update branch December 10, 2024 22:39
mori360 pushed a commit to mori360/pytorch that referenced this pull request Dec 11, 2024
pytorch-linux-jammy-py3-clang12-executorch [started to fail](https://github.com/pytorch/pytorch/actions/runs/12244909721/job/34157668780) today due to a 404 on the Vulkan SDK we use/download (1.2.198.1, 3 years old, URL: https://sdk.lunarg.com/sdk/download/1.2.198.1/linux/vulkansdk-linux-x86_64-1.2.198.1.tar.gz )

The Vulkan SDK is probably no longer needed for building Executorch, and is not used down the line for testing.

This PR tests removing the installation of the SDK

pytorch/executorch#7258
Pull Request resolved: pytorch#142424
Approved by: https://github.com/huydhn
bluenote10 pushed a commit to bluenote10/pytorch that referenced this pull request Dec 14, 2024
pytorch-linux-jammy-py3-clang12-executorch [started to fail](https://github.com/pytorch/pytorch/actions/runs/12244909721/job/34157668780) today due to a 404 on the Vulkan SDK we use/download (1.2.198.1, 3 years old, URL: https://sdk.lunarg.com/sdk/download/1.2.198.1/linux/vulkansdk-linux-x86_64-1.2.198.1.tar.gz )

The Vulkan SDK is probably no longer needed for building Executorch, and is not used down the line for testing.

This PR tests removing the installation of the SDK

pytorch/executorch#7258
Pull Request resolved: pytorch#142424
Approved by: https://github.com/huydhn
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: vulkan Issues related to the Vulkan delegate and code under backends/vulkan/ topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants