Skip to content
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

Add vulkan sample image #27

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

elezar
Copy link
Member

@elezar elezar commented Mar 18, 2025

This change adds a vulkan sample image to replace the image(s) defined at https://gitlab.com/nvidia/container-images/vulkan

$ docker run --rm -ti --runtime=nvidia -e NVIDIA_VISIBLE_DEVICES=runtime.nvidia.com/gpu=all ghcr.io/nvidia/k8s-samples:vulkan-38c01b50-ubuntu22.04
Running headless compute example
GPU: Tesla V100-SXM2-16GB-N
Compute input:
0       1       2       3       4       5       6       7       8       9       10      11      12      13      14      15      16      17      18      19      20      21      22      23      24      25      26 27      28      29      30      31
Compute output:
0       1       1       2       3       5       8       13      21      34      55      89      144     233     377     610     987     1597    2584    4181    6765    10946   17711   28657   46368   75025   121393     196418  317811  514229  832040  1346269
Finished. Press enter to terminate...\n
Running headless rendering example
GPU: Tesla V100-SXM2-16GB-N
Framebuffer image saved to headless.ppm
Finished. Press enter to terminate...\n

@@ -0,0 +1,65 @@
# SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
Copy link
Contributor

@guptaNswati guptaNswati Mar 18, 2025

Choose a reason for hiding this comment

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

what SPDX is?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's a tag to improve machine-readability of the license headers.

See https://spdx.dev/learn/handling-license-info/

libglm-dev \
&& rm -rf /var/lib/apt/lists/*

# TODO: We should update to the official samples
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious, why are we not using the official ones in this build?

Copy link
Member Author

Choose a reason for hiding this comment

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

I did not want to change how the samples are being built when compared to https://gitlab.com/nvidia/container-images/vulkan/-/blame/master/samples/Dockerfile.ubuntu?ref_type=heads#L35 in this commit.

@elezar elezar marked this pull request as draft March 19, 2025 11:35
@elezar elezar force-pushed the add-vulkan-sample branch from 2d80bea to 3df6fd5 Compare March 19, 2025 13:13
@elezar elezar marked this pull request as ready for review March 19, 2025 13:39
@elezar
Copy link
Member Author

elezar commented Mar 19, 2025

cc @dualvtable

elezar added 2 commits March 19, 2025 15:43
This change adds a sample image that can be used to demonstrate
vulkan functionality.

Signed-off-by: Evan Lezar <[email protected]>
Signed-off-by: Evan Lezar <[email protected]>
@elezar elezar force-pushed the add-vulkan-sample branch from 38c01b5 to 72a3b6f Compare March 19, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants