-
Notifications
You must be signed in to change notification settings - Fork 87
Use 25.10 vulkan #208
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
Use 25.10 vulkan #208
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideSimplify Vulkan support by replacing the manual SDK download and build process with the libvulkan1 package and removing SDK environment variable setup in the Dockerfile. Class diagram for updated apt-install.sh script logicclassDiagram
class AptInstall {
+main()
-vulkan_version
-arch
-Manual Vulkan SDK install (removed)
+libvulkan1 install
}
AptInstall : +packages[]
AptInstall : +apt-get update
AptInstall : +apt-get install -y
AptInstall : -wget, tar, build deps (removed)
AptInstall : +libvulkan1
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR simplifies Vulkan installation by replacing custom SDK installation with system package management. It removes the complex custom Vulkan SDK installation process and switches to using the standard libvulkan1 package from the distribution's package manager, which should provide better hardware support and ARM compatibility.
- Simplified Vulkan installation using system packages instead of custom SDK
- Removed custom build process for non-x86_64 architectures
- Cleaned up Docker environment variables related to custom Vulkan SDK
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| scripts/apt-install.sh | Replaced complex custom Vulkan SDK installation with simple libvulkan1 package |
| Dockerfile | Removed environment variables specific to custom Vulkan SDK installation |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Summary of ChangesHello @ericcurtin, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request refactors the Vulkan integration process by moving away from a custom Vulkan SDK download and build approach. Instead, it now relies on the system's package manager to install the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request simplifies the Vulkan setup by switching from a manual SDK installation to using Ubuntu's libvulkan1 package. This is a great improvement that makes the Docker image build process cleaner and more reliable, especially for ARM architectures. My main feedback is to also include the mesa-vulkan-drivers package. This will provide a software-based Vulkan implementation (lavapipe), ensuring that Vulkan can be utilized even without a dedicated GPU, which is important for the 'cpu' and 'generic' variants. Without this, the Vulkan loader won't find any devices to use.
24b37e2 to
cc06430
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
d523367 to
2d6ce4d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
It should have half decent hardware support and supports ARM. Signed-off-by: Eric Curtin <eric.curtin@docker.com>
It should have half decent hardware support and supports ARM.
Summary by Sourcery
Simplify Vulkan support by installing the libvulkan1 package and removing manual SDK download, build logic, and related Dockerfile environment variables
Enhancements:
Build:
Chores: