Skip to content

[Docs] Add SO-101 tutorial banner - #7591

Open
StafaH wants to merge 1 commit into
isaac-sim:developfrom
StafaH:docs/add-so101-tutorial-banner
Open

[Docs] Add SO-101 tutorial banner#7591
StafaH wants to merge 1 commit into
isaac-sim:developfrom
StafaH:docs/add-so101-tutorial-banner

Conversation

@StafaH

@StafaH StafaH commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Description

Add the NVIDIA-hosted SO-101 vial-placement rollout as a banner at the top of the standalone Tutorial page. The 1280 by 720 GIF is displayed at 85% width and remains hosted outside the repository.

Add a short introductory note connecting the tutorial to the NVIDIA Sim-to-Real SO-101 learning course and its workshop repository.

Type of change

  • Documentation update

Release backport

  • Backport this pull request to the active release branch after it merges into develop

Screenshots

The tutorial page now opens with the centered SO-101 vial-placement banner above the workshop note.

Validation

  • uv run --isolated --extra dev -- make -C docs current-docs
  • uv run isaaclab -f
  • Verified the rendered banner width and both workshop links in the generated HTML
  • Verified the hosted GIF returns HTTP 200 with image/gif content

Checklist

  • I have read and understood the contribution guidelines
  • I have run the pre-commit checks with uv run isaaclab -f
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Tests are not required for this documentation-only change
  • No changelog fragment is required because no source package changed
  • My name already exists in CONTRIBUTORS.md

@StafaH
StafaH requested a review from a team September 5, 2026 00:25
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 5, 2026
@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds an NVIDIA-hosted SO-101 vial-placement banner to the standalone tutorial and introduces a note linking readers to the related NVIDIA learning course and workshop repository.

  • Replaces the previous full-width GitHub-hosted demonstration image with a centered 85%-width figure.
  • Adds context connecting the tutorial to the broader Sim-to-Real SO-101 workflow.

Confidence Score: 5/5

The documentation-only change appears safe to merge.

The new figure follows existing documentation conventions, the external links use valid reStructuredText syntax, and no concrete build, rendering, navigation, or security failure remains.

Important Files Changed

Filename Overview
docs/source/setup/tutorial.rst Updates the tutorial introduction with a hosted SO-101 banner and links to related learning resources; no actionable issues found.

Reviews (1): Last reviewed commit: "Add SO-101 tutorial banner" | Re-trigger Greptile

@isaaclab-review-bot isaaclab-review-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isaac Lab Review Bot

Reviewed the tutorial-page media replacement and new SO-101 course/workshop note. The change is confined to documentation and preserves the existing tutorial introduction and navigation structure.

  • Design and architecture: The page now uses a banner-first layout and replaces the raw GitHub-hosted image with an NVIDIA-hosted GIF. The existing page label and grid-based navigation remain unchanged; reliance on an externally hosted asset remains a non-blocking availability tradeoff.
  • API: No source package, public API, CLI, configuration, or reference target is changed. Because this is documentation-only and changes no source package, the repository rules do not require a changelog fragment.
  • Implementation: The new reStructuredText figure includes alternative text, centered alignment, and an explicit width. The course and workshop links use valid anonymous-link syntax, and removal of the prior image block does not alter the surrounding tutorial content or navigation targets.

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.

@StafaH

StafaH commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

run-ci

@isaaclab-bot isaaclab-bot Bot added ci:run-docker Trigger the on-demand Docker and GPU CI workflow and removed ci:run-docker Trigger the on-demand Docker and GPU CI workflow labels Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant