[no-ci] docs: add uv and pixi installation instructions#1650
[no-ci] docs: add uv and pixi installation instructions#1650leofang merged 6 commits intoNVIDIA:mainfrom
Conversation
Add installation instructions for two modern Python package managers: - uv: A fast Python package and project manager from Astral, compatible with PyPI packages. Instructions cover basic install, optional deps, and virtual environment creation. - pixi: A cross-platform package manager built on the conda ecosystem, with support for conda-forge packages. Instructions cover project initialization, adding dependencies via CLI, and pixi.toml config. Both tools are increasingly popular in the scientific Python community and provide faster dependency resolution than traditional pip/conda. Closes NVIDIA#1248
mdboom
left a comment
There was a problem hiding this comment.
In fairness the issue this is response to is pretty minimal, but I think it sort of misses the mark.
A couple obvious issues:
- The
cuda_bindingsinstructions say how to installcuda_python. - The advantage of
pixiis really to create a consistent environment for development, which these instructions don't really address.
I don't think we need to document "how to install a package with your package manager of choice", but it's fine to document "a well-trodden, supported path for making local development easier", and this is kind of neither.
|
Thanks for the detailed feedback — that’s fair. My intent was to document a supported, repeatable path for local development, not 'yet another way to install a package.' I agree the current wording doesn’t make that clear and it overlaps with existing cuda_bindings install docs. I’ll update the PR to:
If you have a preferred structure/location for 'development environment' docs (e.g. CONTRIBUTING / developer guide), I’m happy to align with that. |
|
@mdboom Follow-up with the exact commands/paths (my last comment got eaten by shell backticks):
Commit: 9f296a1 |
|
Hi @mdboom — thanks again for the thorough review. I've addressed all your points in the latest commits:
Could you take another look when you get a chance? Happy to adjust further based on your feedback. |
|
All review feedback has been addressed. This is ready to merge when you have a moment. Thanks! |
Remove changes to cuda_bindings/ as requested by maintainer — external contributors cannot modify files under that path. The uv/pixi dev workflow docs remain in cuda_core/. Signed-off-by: Maxime Grenu <maxime.grenu@gmail.com>
|
Totally fair — apologies for touching the restricted path, should have checked first. Reverted in the latest push; the uv/pixi sections now live solely under cuda_core/. Appreciate the patience on this one, and thanks for the heads-up about the upcoming automation in #1871. Happy to adjust anything else if needed. |
|
/ok to test 9e8ade0 |
|
Thanks @cluster2600 @mdboom ! |
This comment has been minimized.
This comment has been minimized.
1 similar comment
|
What
Add installation instructions for
uvandpixipackage managers to the installation documentation for bothcuda.bindingsandcuda.core.Why
Closes #1248. Both
uvandpixiare gaining significant adoption in the scientific Python and CUDA communities:Users actively request these instructions, and providing them lowers the barrier to entry for developers using modern tooling.
How
Added new sections in:
cuda_bindings/docs/source/install.rst– "Installing with uv" and "Installing with pixi" sections after the Conda sectioncuda_core/docs/source/install.rst– same sections, adapted forcuda-core's CUDA-version-specific extras ([cu12],[cu13])Each section includes:
Testing
docs.astral.sh/uv,pixi.sh) are live and correctChecklist
cuda.bindingsandcuda.coreuvandpixi#1248