Skip to content

Humanize concepts docs#526

Open
alexmillane wants to merge 42 commits intomainfrom
alex/docs/humanize_concepts
Open

Humanize concepts docs#526
alexmillane wants to merge 42 commits intomainfrom
alex/docs/humanize_concepts

Conversation

@alexmillane
Copy link
Copy Markdown
Collaborator

@alexmillane alexmillane commented Apr 1, 2026

Summary

Rework the concepts documentation to eliminate AI slop.

@cvolkcvolk cvolkcvolk force-pushed the cvolk/first-experiments-gr00t-docs branch from 55b9269 to 02df632 Compare April 1, 2026 14:22
@cvolkcvolk cvolkcvolk changed the base branch from cvolk/first-experiments-gr00t-docs to main April 7, 2026 11:13
@cvolkcvolk cvolkcvolk force-pushed the alex/docs/humanize_concepts branch from f83ac21 to f92ea6b Compare April 7, 2026 11:27
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
… page

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Replace AI-generated bullet lists and abstract stubs with a
walkthrough of the two-phase recording system. Use the real
ObjectMovedRateMetric implementation as the custom metric example.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Move concept_policy_design, concept_remote_policies_design, and
policy_evaluation/evaluation_types into a new concepts/policy/
subdirectory. Remove the standalone Policy Evaluation section from
the top-level navigation. Update all cross-references.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
@cvolkcvolk cvolkcvolk marked this pull request as ready for review April 8, 2026 12:09
cvolkcvolk and others added 4 commits April 8, 2026 14:12
Resolve conflict in docs/index.rst: keep Concepts toctree from branch,
add new Arena in Your Repo section from main. Move Arena in Your Repo
above Concepts in the navigation.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
@alexmillane alexmillane enabled auto-merge (squash) April 8, 2026 16:40
Copy link
Copy Markdown

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

Choose a reason for hiding this comment

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

Nice cleanup — the rewritten pages read much better than the old templated format. A few typos and minor wording issues I spotted below.

cvolkcvolk and others added 10 commits April 9, 2026 13:27
Co-authored-by: isaaclab-review-bot[bot] <270793704+isaaclab-review-bot[bot]@users.noreply.github.com>
Co-authored-by: isaaclab-review-bot[bot] <270793704+isaaclab-review-bot[bot]@users.noreply.github.com>
Co-authored-by: isaaclab-review-bot[bot] <270793704+isaaclab-review-bot[bot]@users.noreply.github.com>
Replace nonexistent how_open() with get_openness() to match the
actual Openable affordance API.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Co-authored-by: isaaclab-review-bot[bot] <270793704+isaaclab-review-bot[bot]@users.noreply.github.com>
Co-authored-by: isaaclab-review-bot[bot] <270793704+isaaclab-review-bot[bot]@users.noreply.github.com>
Copy link
Copy Markdown

@kellyguo11 kellyguo11 left a comment

Choose a reason for hiding this comment

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

Nice work cleaning up the docs — the rewritten pages are a massive improvement over the old templated format. The restructuring into scene/, task/, embodiment/, and policy/ subdirectories makes the concept docs much more navigable.

A few things I spotted:

@cvolkcvolk cvolkcvolk disabled auto-merge April 10, 2026 12:59
cvolkcvolk and others added 7 commits April 10, 2026 15:03
Reset submodules/IsaacLab back to the commit on main
(e57379c6). It was inadvertently changed in b4311e5.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Superpowers plans and specs are now stored outside the repo,
so the gitignore entry is no longer needed.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
Signed-off-by: Clemens Volk <cvolk@nvidia.com>
RST requires a blank line before bullet lists for correct parsing.

Signed-off-by: Clemens Volk <cvolk@nvidia.com>
@xyao-nv xyao-nv enabled auto-merge (squash) April 10, 2026 20:11
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.

4 participants