[DOC] Clarify uv extra synchronization behavior - #7148
Conversation
There was a problem hiding this comment.
Isaac Lab Review Bot
Documentation-only change centralizing uv extra-synchronization guidance under the new installation-uv-extra-sync label and linking relevant installation, tutorial, overview, showroom, and troubleshooting pages to it. The RSL-RL troubleshooting command now preserves previously selected extras with uv sync --inexact.
- Design and architecture: The single-source guidance avoids duplicating exact-versus-inexact synchronization details. The new references remain correctly nested within their surrounding reStructuredText notes and tab items, and the target label is defined once.
- API: No source APIs, CLI arguments, dependencies, or public symbols change. The explicit text on each
:ref:is appropriate for a label attached to a paragraph rather than a titled section. - Implementation: The documented behavior is consistent with uv semantics:
uv run --extraminimally updates the environment,uv sync --inexact --extrapreserves unrelated installed packages, and plain exactuv synccan remove packages belonging only to omitted extras. The troubleshooting update follows that guidance. The showroom link is slightly broader than its immediateuv runexample but remains relevant and non-blocking.
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.
Greptile SummaryThe PR clarifies how optional uv extras interact with exact and inexact synchronization.
Confidence Score: 5/5The documentation-only PR appears safe to merge with no actionable defects identified. The new synchronization guidance is consistently referenced from the affected pages, and the added global documentation target is present in the installation guide. Important Files Changed
Reviews (1): Last reviewed commit: "Clarify uv extra synchronization guidanc..." | Re-trigger Greptile |
StafaH
left a comment
There was a problem hiding this comment.
This is too complicated for the typical user. We don't want them to use uv sync --inexact. Why can't they just use the extra?
@StafaH You mean |
f5b0a22 to
0f6aa77
Compare
Description
This follow-up to #7096 clarifies how uv handles optional extras across installation, tutorial, overview, and troubleshooting documentation.
The installation guide now explains when to use
uv run --extraversusuv sync --inexact --extra, and warns that plainuv syncperforms an exact sync that removes packages belonging only to omitted extras. The affected tutorial and overview pages link to this guidance, and troubleshooting now usesuv sync --inexactso it does not remove previously selected extras.No additional dependencies are required.
Type of change
Screenshots
Not applicable.
Checklist
pre-commitchecks withuv run isaaclab -fCONTRIBUTORS.mdor my name already exists there