ENH: Pre-installing setuptools in nightly-health - #101
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Cache: Disabled due to data retention organization setting Knowledge base: Disabled due to data retention organization setting WalkthroughThe nightly health workflow installs ChangesNightly health workflow
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the nightly health GitHub Actions workflow to ensure the build toolchain is present in the virtual environment before installing CUDA/PhysicsNeMo-related dependencies, aligning with the project’s pyproject.toml guidance around torch-scatter and --no-build-isolation.
Changes:
- Adds an explicit
setuptools/wheelinstallation step prior to installing.[cuda13]and.[physicsnemo]extras. - Documents why the install ordering matters for
torch-scatterbuilds in uv-managed environments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by CodeRabbit