Enhance provisioning scripts to include Pixi installation - #6
Merged
Conversation
volpatto
commented
Aug 18, 2026
volpatto
commented
Member
- Updated README.md to reflect the addition of Pixi installation across Ubuntu, macOS, and Windows provisioning scripts.
- Modified Ubuntu provisioning script to install Pixi for the invoking user, with CLI flags to control installation.
- Enhanced macOS provisioning script to include Pixi installation via Homebrew, with appropriate flags.
- Updated Windows PowerShell script to install Pixi for the current user, with verification of installation.
- Adjusted CUDA and NVIDIA driver installation logic to clarify independence and requirements.
- Updated README.md to reflect the addition of Pixi installation across Ubuntu, macOS, and Windows provisioning scripts. - Modified Ubuntu provisioning script to install Pixi for the invoking user, with CLI flags to control installation. - Enhanced macOS provisioning script to include Pixi installation via Homebrew, with appropriate flags. - Updated Windows PowerShell script to install Pixi for the current user, with verification of installation. - Adjusted CUDA and NVIDIA driver installation logic to clarify independence and requirements.
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Pixi installation support to the existing cross-platform provisioning scripts (Ubuntu/macOS/Windows), while also clarifying and decoupling NVIDIA driver vs CUDA Toolkit setup to better match typical PyTorch workstation needs.
Changes:
- Added Pixi installation (with toggle flags) to Ubuntu, macOS (Homebrew), and Windows (PowerShell) provisioning scripts.
- Refactored Ubuntu/macOS CLI flag handling so explicit flags override profile defaults deterministically.
- Updated CUDA/NVIDIA driver logic and user-facing documentation to emphasize driver/toolkit independence.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| scripts/provision-win.ps1 | Adds optional Pixi install and improves Miniconda installer error handling; adjusts CUDA/driver logic. |
| scripts/provision-ubuntu.sh | Adds user-scoped Pixi install, introduces NVIDIA repo/driver logic, and refines profile/flag precedence. |
| scripts/provision-macos.sh | Adds Pixi via Homebrew, refines profile/flag precedence, and blocks sudo/root execution. |
| README.md | Documents Pixi installation and the updated NVIDIA/CUDA guidance and flags across platforms. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+261
to
+264
| $installer = Join-Path $env:TEMP 'install-pixi.ps1' | ||
| Write-Host "Downloading the official Pixi installer..." | ||
| Invoke-WebRequest -Uri 'https://pixi.sh/install.ps1' -OutFile $installer -UseBasicParsing | ||
| & powershell.exe -NoLogo -NoProfile -ExecutionPolicy Bypass -File $installer |
Comment on lines
+262
to
+267
| Write-Host "Downloading the official Pixi installer..." | ||
| Invoke-WebRequest -Uri 'https://pixi.sh/install.ps1' -OutFile $installer -UseBasicParsing | ||
| & powershell.exe -NoLogo -NoProfile -ExecutionPolicy Bypass -File $installer | ||
| $pixiExitCode = $LASTEXITCODE | ||
| Remove-Item $installer -Force -ErrorAction SilentlyContinue | ||
| if ($pixiExitCode -ne 0) { throw "Pixi installer failed with exit code $pixiExitCode." } |
Comment on lines
+237
to
241
| if [ "$INSTALL_VSCODE" = "true" ]; then | ||
| setup_code_repo | ||
| fi | ||
| log "Updating apt and installing base packages..." | ||
| apt-get update |
Comment on lines
+285
to
+294
| if [ "$INSTALL_DRIVER" = "true" ]; then | ||
| if $IS_WSL; then | ||
| warn "Skipping Linux NVIDIA driver in WSL2; install/update the NVIDIA driver on Windows." | ||
| else | ||
| log "Installing NVIDIA driver package: $NVIDIA_DRIVER_PKG" | ||
| DEBIAN_FRONTEND=noninteractive apt-get install -y \ | ||
| "linux-headers-$(uname -r)" "$NVIDIA_DRIVER_PKG" | ||
| REBOOT_REQUIRED=true | ||
| fi | ||
| fi |
Comment on lines
+354
to
+358
| installer="$(mktemp /tmp/pixi-install.XXXXXX.sh)" | ||
| curl -fsSL https://pixi.sh/install.sh -o "$installer" | ||
| group="$(id -gn "$u")" | ||
| chown "$u":"$group" "$installer" | ||
| sudo -H -u "$u" env HOME="$h" bash "$installer" |
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.