Update Hugging Face CLI commands - #7665
Merged
Merged
Conversation
jaybdub
requested review from
AntoineRichard,
StafaH,
kellyguo11 and
ooctipus
as code owners
September 9, 2026 03:09
github-actions
Bot
added
bug
Contributor
 The documentation-only CLI update appears safe to merge.
|
![isaaclab-review-bot[bot]](https://avatars.githubusercontent.com/u/157846462?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Isaac Lab Review Bot
The docs-only change updates both G1 locomanipulation download examples from the deprecated huggingface-cli download entry point to hf download without changing repositories, destinations, or workflow steps.
- Design and architecture: The change remains limited to two existing Bash blocks and preserves the documented download → unzip → finetune/rollout workflow. It introduces no new dependencies or documentation structure.
- API: No Isaac Lab public API is affected. The documented Hugging Face CLI arguments, repository IDs, repository type, and local output directories remain unchanged; only the CLI executable name changes.
- Implementation: Both updated commands remain consistent with their subsequent wildcard
unzipcommands and documented extraction paths. Users need a sufficiently recenthuggingface_hubinstallation that provideshf, but that compatibility requirement is inherent to replacing the deprecated command and does not demonstrate a defect in this focused update.
No blocking issues. No inline issue met the actionable-evidence threshold; the assessment above records the review feedback.
Automated review; human maintainers own approval decisions.
jaybdub
force-pushed
the
nvbug/6682479
branch
from
September 9, 2026 19:50
827f66c to
155aacc
Compare
jaybdub
requested review from
ClemensSchwarke,
Toni-SM,
aserifi,
david-cao-mueller,
fatimaanes,
frlai,
hougantc-nvda,
huidongc,
hujc7,
marcodiiga,
matthewtrepte,
myurasov-nv,
pbarejko,
peterd-NV,
r-schmitt,
rilei-nvidia,
rubengrandia and
rwiltz
as code owners
September 9, 2026 19:50
github-actions
Bot
added
asset
kellyguo11
approved these changes
Sep 9, 2026
Contributor
|
run-ci |
isaaclab-bot
Bot
added
ci:run-docker
Contributor
|
run-ci |
isaaclab-bot
Bot
added
ci:run-docker
Contributor
|
Backported to |
isaaclab-bot Bot
pushed a commit
that referenced
this pull request
Sep 9, 2026
# Description Replaces the deprecated `huggingface-cli download` examples with the current `hf download` command for the G1 locomanipulation dataset and checkpoint. Tracks NVBug 6682479. ## Type of change - Documentation update ## Release backport - [x] <!-- backport-active-release --> Backport this pull request to the active release branch after it merges into `develop` ## Validation - `uv run --isolated --extra test -- make -C docs current-docs` - `uv run isaaclab -f` - Used both documented `hf download` forms to fetch a small file from the real dataset and model repositories. ## Screenshots Not applicable. ## Checklist - [x] I have read and understood the contribution guidelines. - [x] I have run the pre-commit checks. - [x] I have made the corresponding documentation change. - [x] My changes generate no new warnings. - [x] I have run targeted validation for the fix. - [x] Changelog fragment is not applicable to this docs-only change. - [x] The contributor already appears in repository revision history. Co-authored-by: Kelly Guo <kellyg@nvidia.com> (cherry picked from commit 20ef8d7)
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
2 participants
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.
Description
Replaces the deprecated
huggingface-cli downloadexamples with the currenthf downloadcommand for the G1 locomanipulation dataset and checkpoint.Tracks NVBug 6682479.
Type of change
Release backport
developValidation
uv run --isolated --extra test -- make -C docs current-docsuv run isaaclab -fhf downloadforms to fetch a small file from the real dataset and model repositories.Screenshots
Not applicable.
Checklist