Skip to content

Inspect Exec Connections#48

Draft
chrizzFTD wants to merge 7 commits into
developfrom
feature/exec_connections_inspection
Draft

Inspect Exec Connections#48
chrizzFTD wants to merge 7 commits into
developfrom
feature/exec_connections_inspection

Conversation

@chrizzFTD

@chrizzFTD chrizzFTD commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator

Allow inspection of exec networks through the ConnectionViewer


Note

Medium Risk
Touches shared connection-graph code used for UsdShade and new attribute traversal; wrong routing or port IDs could break graph layout or misrepresent rig wiring, though existing shade tests remain and exec coverage is additive.

Overview
Extends the Connections Viewer so rigs and other prims wired through UsdAttribute connection paths (not only UsdShadeConnectableAPI) show an upstream authored graph—covering OpenExec invertible rigs (IrJointScope, IrFkController, IrSwitchController, etc.).

_graph_from_connections now picks shade vs attribute traversal. Attribute mode walks GetConnections(), builds Graphviz nodes with schema type in the header, maps irRole to port colors (inputs, outputs, switches, passthrough), and sanitizes port IDs when names contain : (e.g. posed:spaceposed__space). UsdShade graph building shares the same table/port helpers.

Docs for the Connections Viewer describe the dual traversal model, port legend, and that the view is the authored stage graph, not the compiled OpenExec eval network. Adds waddlerRig.usda and test_exec_connection_view to lock in Switch/FK wiring from a joint prim.

Reviewed by Cursor Bugbot for commit 2a3509e. Bugbot is set up for automated code reviews on this repo. Configure here.

@chrizzFTD

Copy link
Copy Markdown
Collaborator Author

@cursor review

@cursor cursor 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 2a3509e. Configure here.

chrizzFTD added 5 commits June 8, 2026 11:53
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
@chrizzFTD chrizzFTD force-pushed the feature/exec_connections_inspection branch from 6af2f4a to 452e454 Compare June 8, 2026 01:54
chrizzFTD added 2 commits June 8, 2026 11:58
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
Signed-off-by: Christian López Barrón <chris.gfz@gmail.com>
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.

1 participant