Skip to content

fix(pixi): restore conda test deps and relock after #2384 - #2532

Merged
juenglin merged 1 commit into
NVIDIA:mainfrom
juenglin:test-helpers-postfix
Aug 10, 2026
Merged

fix(pixi): restore conda test deps and relock after #2384#2532
juenglin merged 1 commit into
NVIDIA:mainfrom
juenglin:test-helpers-postfix

Conversation

@juenglin

@juenglinjuenglin commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix #2377: Reorganize the test helpers (where appropriate) to cuda_python_test_helpers #2384 inserted [feature.test.pypi-dependencies] mid-table in cuda_bindings/cuda_corepixi.toml, accidentally moving conda test deps (pytest-benchmark, numpy, pyglet, …) to PyPI and leaving the lockfiles stale.
  • Fresh CI installs then dropped the local cuda-bindings/cuda-core source packages, so the pixi source-build smoke failed with ModuleNotFoundError: No module named 'cuda'.
  • Restore those deps under [feature.test.dependencies], keep only cuda-python-test-helpers under pypi, and regenerate both lockfiles with Pixi 0.73 until pixi lock --check is clean.

Test plan

  • pixi lock --check --manifest-path cuda_bindings
  • pixi lock --check --manifest-path cuda_core
  • Re-run CI: pixi run test (source build) (build smoke) on this branch and confirm bindings import OK / core import OK

NVIDIA#2384 inserted a pypi-dependencies header mid-table, moving conda test
deps to PyPI without updating lockfiles. Fresh CI installs then dropped
the local cuda-bindings/cuda-core source packages, causing ModuleNotFoundError.
@juenglinjuenglin added this to the cuda.bindings next milestone Aug 7, 2026
@juenglinjuenglin added bug Something isn't working P1 Medium priority - Should do CI/CD CI/CD infrastructure labels Aug 7, 2026
@copy-pr-bot

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actionsgithub-actionsBot added cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module labels Aug 7, 2026
@juenglin
juenglin marked this pull request as ready for review August 7, 2026 20:34
@juenglinjuenglin self-assigned this Aug 7, 2026
@github-actions

This comment has been minimized.

@rwgkrwgk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @juenglin! (sorry I didn't see this before in the ever swelling flood of emails)

@juenglin
juenglin merged commit 4b13910 into NVIDIA:mainAug 10, 2026
131 of 132 checks passed
@github-actions

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

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

Labels

bugSomething isn't workingCI/CDCI/CD infrastructurecuda.bindingsEverything related to the cuda.bindings modulecuda.coreEverything related to the cuda.core moduleP1Medium priority - Should do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@juenglin@rwgk