Conversation
PEP 508 extra currently requests Isaac-GR00T @ git+..., but NVIDIA/Isaac-GR00T declares name = "gr00t". uv/pip then fail with: Package metadata name `gr00t` does not match given name `Isaac-GR00T` Use the declared package name. Reported on NVIDIA/Isaac-GR00T#748. Signed-off-by: John Wantz Jr. <wantzjt@gmail.com>
Isaac-GR00T requires-python is >=3.12,<3.13. install_inference.sh was still pinning uv-managed 3.10, so gear_sonic[inference] cannot install even after the package-name fix. Leave other install_* scripts on 3.10; they do not pull Isaac-GR00T. Signed-off-by: John Wantz Jr. <wantzjt@gmail.com>
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.
Problem
bash install_scripts/install_inference.shfails while installinggear_sonic[inference]:That is the failure reported on NVIDIA/Isaac-GR00T#748. The bug is in this repo, not in Isaac-GR00T.
Current extra:
"Isaac-GR00T @ git+https://github.com/NVIDIA/Isaac-GR00T.git"Current Isaac-GR00T
pyproject.toml:Even after the name fix,
install_inference.shstill creates a Python 3.10 venv, which Isaac-GR00T will reject.Change
gear_sonic/pyproject.toml— depend ongr00t @ git+https://github.com/NVIDIA/Isaac-GR00T.git(PEP 508 name matches package metadata).install_scripts/install_inference.sh— install uv-managed Python 3.12 for.venv_inferenceonly.Other
install_*scripts stay on 3.10. They do not pull Isaac-GR00T.Why this is scoped this way
gear_sonicstill declaresrequires-python = ">=3.10"; 3.12 is compatible.Validation
install_inference.shhere (downloads Isaac-GR00T + CUDA stack). Happy to iterate if CI or a maintainer wants a different pin (commit SHA vsgit+...git).Fixes the install path described in NVIDIA/Isaac-GR00T#748.
Signed-off-by: John Wantz Jr.