Skip to content

Conversation

@bdice
Copy link
Contributor

@bdice bdice commented Dec 5, 2025

Description

This PR removes pre-release upper bound pinnings from non-RAPIDS dependencies.

The presence of pre-release indicators like <...a0 tells pip "pre-releases are OK, even if --pre was not passed to pip install." RAPIDS projects currently use such constraints in situations where it's not actually desirable to get pre-releases.

xref: rapidsai/build-planning#144

Summary by CodeRabbit

  • Chores
    • Updated dependency bounds across environments and project configs to use stable release upper limits (removed prerelease/dev suffixes). Affected packages include rapids-build-backend, numpy, cuda-python, and numba-cuda to ensure compatibility with final releases.

✏️ Tip: You can customize this high-level summary in your review settings.

@bdice bdice requested a review from a team as a code owner December 5, 2025 18:11
@bdice bdice requested a review from gforsyth December 5, 2025 18:11
@bdice bdice added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Dec 5, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 5, 2025

📝 Walkthrough

Walkthrough

Removed prerelease suffixes (a0, .dev0) from upper-bound version constraints for several dependencies (rapids-build-backend, cuda-python, numba-cuda, numpy) across conda environments, conda recipes, and Python pyproject files.

Changes

Cohort / File(s) Summary
Conda Environment Files
conda/environments/all_cuda-129_arch-aarch64.yaml, conda/environments/all_cuda-129_arch-x86_64.yaml, conda/environments/all_cuda-130_arch-aarch64.yaml, conda/environments/all_cuda-130_arch-x86_64.yaml
Removed prerelease suffixes from upper bounds: cuda-python (<13.0a0→<13.0 or <14.0a0→<14.0), numba-cuda (<0.20.0a0→<0.20.0), numpy (<3.0a0→<3.0), rapids-build-backend (<0.5.0.dev0→<0.5.0)
Conda Recipe Files
conda/recipes/cuopt-server/recipe.yaml, conda/recipes/cuopt/recipe.yaml, conda/recipes/mps-parser/recipe.yaml
Updated host/build/run requirements to remove prerelease qualifiers: rapids-build-backend <0.5.0.dev0→<0.5.0; tightened cuda-python, numba-cuda, and numpy upper bounds where present
Python Project Configs
python/cuopt/cuopt/linear_programming/pyproject.toml, python/cuopt/pyproject.toml, python/cuopt_server/pyproject.toml, python/libcuopt/pyproject.toml
Aligned build-system and project dependency bounds to stable releases by removing .dev0/a0 suffixes for rapids-build-backend, cuda-python, numba-cuda, and numpy
Dependency Manifest
dependencies.yaml
Unified dependency pins across matrices by removing prerelease qualifiers for rapids-build-backend, cuda-python (12.9/13.0 variants), numba-cuda, and numpy

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Repetitive, consistent pattern changes (removal of prerelease suffixes) across files
  • Attention points:
    • Verify consistency of updated upper bounds across CUDA matrix variants (12.9 vs 13.0/14.0)
    • Confirm no occurrences of the old prerelease suffixes remain
    • Check that caret/other conditional branches in recipe files were updated uniformly

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'Remove alpha specs from non-RAPIDS dependencies' accurately summarizes the main change: removing pre-release specifiers (a0, dev0) from dependency constraints across multiple configuration files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 604a72f and ac62225.

📒 Files selected for processing (1)
  • dependencies.yaml (4 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • dependencies.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.11, amd64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.12, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.11, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.12, amd64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.13, arm64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.13, amd64, rockylinux8
  • GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.10, amd64, rockylinux8

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gforsyth
Copy link
Contributor

gforsyth commented Dec 5, 2025

/merge

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

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants