Skip to content

build: use PyPI nvidia-physicsnemo instead of git source#899

Merged
NickGeneva merged 1 commit into
NVIDIA:mainfrom
NickGeneva:ngeneva/psn_install_update
Jun 2, 2026
Merged

build: use PyPI nvidia-physicsnemo instead of git source#899
NickGeneva merged 1 commit into
NVIDIA:mainfrom
NickGeneva:ngeneva/psn_install_update

Conversation

@NickGeneva
Copy link
Copy Markdown
Collaborator

Remove the git source override for nvidia-physicsnemo in pyproject.toml, allowing the package to be installed from PyPI using the version constraints defined in optional dependencies.

Changes

  • Removed git source entry for nvidia-physicsnemo pointing to commit 369a1578
  • Package will now install from PyPI (v2.1.0 instead of v2.1.0a0)
  • Updated lock file accordingly

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 2, 2026

Greptile Summary

Drops the temporary git-source override for nvidia-physicsnemo (which pinned a pre-release alpha commit to work around a torch 2.11 incompatibility) and lets the package install from PyPI at its stable 2.1.0 release.

  • pyproject.toml: Removes the nvidia-physicsnemo = { git = … } source entry and its TODO comment; version resolution now falls through to the existing >=1.0.1 / >=1.3.0 / >=2.0 specifiers already declared in the optional-dependency groups.
  • uv.lock: Regenerated to reflect the PyPI source, switching from 2.1.0a0 (git) to 2.1.0 (wheel). Transitive bumps also land: gitpython 3.1.46 → 3.1.50, urllib3 2.6.3 → 2.7.0, warp-lang 1.11.1 → 1.14.0.

Confidence Score: 5/5

Straightforward dependency cleanup with no logic changes; the stable PyPI wheel replaces a temporary pre-release git pin.

The only change is moving nvidia-physicsnemo from a pinned pre-release git commit to the stable 2.1.0 PyPI wheel. The TODO comment that motivated the temporary pin is removed together with the override, which is the expected cleanup pattern. Transitive bumps (gitpython, urllib3, warp-lang) are minor and unrelated to the core change. No application logic is touched.

No files require special attention.

Important Files Changed

Filename Overview
pyproject.toml Removes the git source override for nvidia-physicsnemo (pointing to pre-release commit 369a1578) and its accompanying TODO comment, allowing the package to resolve from PyPI.
uv.lock Lock file regenerated: nvidia-physicsnemo moves from git-sourced 2.1.0a0 to PyPI 2.1.0; also picks up gitpython 3.1.50, urllib3 2.7.0, and warp-lang 1.14.0 as transitive bumps.

Reviews (1): Last reviewed commit: "build: use PyPI nvidia-physicsnemo inste..." | Re-trigger Greptile

@NickGeneva
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@NickGeneva NickGeneva requested a review from pzharrington June 2, 2026 06:34
@NickGeneva NickGeneva merged commit e735789 into NVIDIA:main Jun 2, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants