Skip to content

fix(pathfinder): find cuobjdump in the nvidia/cu13/bin wheel layout - #2739

Merged
rwgk merged 1 commit into
NVIDIA:mainfrom
brandon-b-miller:bbm/pathfinder-cuobjdump-cu13
Aug 31, 2026
Merged

fix(pathfinder): find cuobjdump in the nvidia/cu13/bin wheel layout#2739
rwgk merged 1 commit into
NVIDIA:mainfrom
brandon-b-miller:bbm/pathfinder-cuobjdump-cu13

Conversation

@brandon-b-miller

Copy link
Copy Markdown
Contributor

Closes #2738

@github-actions github-actions Bot added the cuda.pathfinder Everything related to the cuda.pathfinder module label Aug 31, 2026
@brandon-b-miller
brandon-b-miller requested a review from rwgk August 31, 2026 15:12
@rwgk rwgk added the P0 High priority - Must do! label Aug 31, 2026
@rwgk rwgk added this to the cuda.pathfinder next milestone Aug 31, 2026
@github-actions

This comment has been minimized.

@rwgk

rwgk commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

This PR made me wonder, are we obviously missing any other existing wheels? I asked codex to look systematically on PyPI. It found:


Among the existing SITE_PACKAGES_BINDIRS entries, no — cuobjdump is the only missing _CUDA13_BIN case. Brandon’s production change is correct.

I inspected the current stable CUDA 13 component wheels for Linux x86-64, Linux ARM64, and Windows x86-64; their SHA-256 hashes matched PyPI metadata:

Those seven utilities live under nvidia/cu13/bin, exactly matching the new test. nvprune, cuda-gdb, cuda-gdbserver, and nvprof are absent from the relevant PyPI wheels. The Nsight commands have separate configured namespaces and are not part of the cuda-toolkit==13.3.1 wheel closure.

One adjacent finding: ptxas is present under both the CUDA 13 and legacy NVCC wheel layouts, but it is not supported by SITE_PACKAGES_BINDIRS at all. That is a separate API-expansion question, rather than another missing _CUDA13_BIN tuple. It also makes the test docstring at test_find_nvidia_binaries.py slightly too broad: “Every supported utility shipped…” would be literally accurate.


I'll make another PR to capture this finding.

@rwgk
rwgk merged commit 3373ae2 into NVIDIA:main Aug 31, 2026
211 of 215 checks passed
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.pathfinder Everything related to the cuda.pathfinder module P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Missing cuda 13 SITE_PACKAGES_BINDIRS entry for cuobjdump wheel paths

2 participants