Skip to content

Follow the reorganized example folders and unified command-line flags. - #138

Open
Milotrince wants to merge 4 commits into
mainfrom
examples-layout-and-flags
Open

Follow the reorganized example folders and unified command-line flags.#138
Milotrince wants to merge 4 commits into
mainfrom
examples-layout-and-flags

Conversation

@Milotrince

Copy link
Copy Markdown
Member

Companion to the examples cleanup in genesis-world, which groups every example under a feature folder and
puts all of them on one command-line convention. Without this, the affected links 404 and the copy-pasteable
commands silently stop working.

Paths that moved

Was Now
examples/IPC_Solver/ examples/ipc/
examples/smoke.py examples/fluid/smoke.py
examples/differentiable_push.py examples/deformable/differentiable_push.py
examples/ddp_multi_gpu.py examples/rigid/ddp_multi_gpu.py

Flags that changed

Long option names now use dashes between words, and the environment count is -b everywhere (it was -B in
the training guides):

  • -B 8192 -> -b 8192
  • --max_iterations -> --max-iterations
  • -B/--num_envs -> -b/--num-envs
  • -e/--exp_name -> -e/--exp-name
  • --use_force -> --use-force

Python attribute names are unaffected (argparse maps dashes to underscores), so the embedded code excerpts
that read args.steps, args.num_envs and args.vis stay correct as written.

Verification

Every examples/... path referenced anywhere under source/ was resolved against the reorganized tree: all
73 file links and all 13 directory links point at something that exists. The two unresolved hits,
01_hello_nyx.py and 05_gaussian_splat.py, live in genesis-nyx and are out of scope.

Merge order

This should land after the genesis-world PR, since the links target blob/main/examples/... at the new
locations.

Copilot AI review requested due to automatic review settings July 25, 2026 00:13

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.

Tip: disable this comment in your organization's Code Review settings.

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The directory list skipped five folders, including the new 'deformable/' and 'fluid/'. The batch-render page
named an output directory the example never writes, and the keybinding snippet stopped importing 'KeyMod'
after the module alias was dropped, though the prose still uses it.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.

Tip: disable this comment in your organization's Code Review settings.

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.

2 participants