Uh oh!
There was an error while loading. Please reload this page.
Enable runtime resolution of CUDA header path for NVRTC - #3252
Conversation
fheinecke
commented
Jul 24, 2026
/te-ci |
fheinecke
commented
Aug 3, 2026
Failing test is due to runner scheduling, should be unrelated to this |
fheinecke
commented
Aug 3, 2026
/te-ci |
Greptile SummaryThe PR adds runtime discovery of CUDA headers for NVRTC while retaining the existing configured and build-time fallbacks.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the prior legacy and Python override issues are addressed by runtime-major-aware selection, and discovered paths are validated before use. Important Files Changed
Reviews (4): Last reviewed commit: "fix linter failure" | Re-trigger Greptile |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Signed-off-by: Fred Heinecke <fheinecke@nvidia.com>
for more information, see https://pre-commit.ci
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> Signed-off-by: fheinecke <23390735+fheinecke@users.noreply.github.com>
… of site-packages directory Signed-off-by: Fred Heinecke <fheinecke@nvidia.com>
Signed-off-by: Fred Heinecke <fheinecke@nvidia.com>
Signed-off-by: Fred Heinecke <fheinecke@nvidia.com>
d11bc9e to
c443baaCompareptrendx
commented
Aug 11, 2026
/te-ci |
Uh oh!
There was an error while loading. Please reload this page.
Description
NVRTC relies on having CUDA toolkit headers available at runtime for JIT compilation. The following directories are searched:
${NVTE_CUDA_INCLUDE_DIR}${CUDA_HOME}${CUDA_DIR}/usr/local/cudaThe build-time path may coincidentally line up with the runtime path, but may be incorrect when building TE on another machine or environment, or when building TE with build isolation enabled. This enables searching site-packages at runtime for the headers, allowing them to be resolved in more environments. If checked paths don't exist, then this will fall back to the existing behavior.
Type of change
Changes
Please list the changes introduced in this PR:
Checklist: