Publish cuda-pathfinder to the torch CUDA arch indexes - #8515
Merged
Conversation
torch declares cuda-bindings, and cuda-bindings requires
cuda-pathfinder>=1.4.2. Release smoke tests install from a single arch index
with no PyPI fallback, so that transitive dependency has to be resolvable
there too.
cuda-pathfinder was registered under the vllm project only, so
update_dependencies.py never wrote whl/{test,nightly}/cu*/cuda-pathfinder/.
whl/test/cu134 advertises it in the PEP 503 listing while the target 403s. The
older arches still serve it, but that is residue from before it stopped being
produced -- nothing regenerates it today, so cu134 is simply the first arch
created after that.
Give it the same torch targets as cuda-bindings, since it exists to satisfy
cuda-bindings. cuda-python is left alone: torch never depends on it, so its
absence from the arch indexes is correct.
|
@atalman is attempting to deploy a commit to the Meta Open Source Team on Vercel. A member of the Team first needs to authorize it. |
tinglvv
approved these changes
Aug 12, 2026
zxiiro
approved these changes
Aug 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://download.pytorch.org/whl/test/cu134/cuda-pathfinder/returns 403 while the cu134 index advertises it:Why it is needed
torch declares
cuda-bindingsinPYTORCH_EXTRA_INSTALL_REQUIREMENTS, andcuda-bindingsin turn requires:Release smoke tests install from a single arch index with no PyPI fallback (
Looking in indexes: https://download.pytorch.org/whl/test/cu134), so a transitive dependency of a declared dependency has to be resolvable in that same index.Why it is missing
cuda-pathfinderwas registered under the vllm project only:so
update_dependencies.pynever writeswhl/{test,nightly}/cu*/cuda-pathfinder/. Confirmed against a live run (job) — it uploaded the identical 18 packages to cu126, cu130, cu132 and cu134, with neither cuda-pathfinder nor cuda-python among them:The older arches still serve
cuda-pathfinder, but that is residue from before it stopped being produced — nothing regenerates it today, which is why cu134, the newest arch, is the one that broke.The change
Give
cuda-pathfinderthe same torch targets ascuda-bindings(cu126, cu128, cu129, cu130, cu132, cu134), keeping the existing vllm entry. It exists to satisfycuda-bindings, so it should follow it exactly.cuda-pythonis deliberately left alone. torch never depends on it — it is vllm-only — so its absence from the torch arch indexes is correct, and its 403 on cu134 is harmless for torch smoke tests. Worth noting the cu134 listing still advertises it, so that dangling link remains; it just is not on the path to anything torch installs.