Skip to content

web: group backside layers under a Backside category in the layer tree#10625

Merged
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:web-backside-layer-category
Jun 9, 2026
Merged

web: group backside layers under a Backside category in the layer tree#10625
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:web-backside-layer-category

Conversation

@maliberty

Copy link
Copy Markdown
Member

Backside layers are split out of each chip node's layers array in the layer_hierarchy JSON and nested in a synthetic child node of type "category". The frontend skips chipletPath for category nodes so they do not participate in chiplet visibility sync.

image

Backside layers are split out of each chip node's layers array in the
layer_hierarchy JSON and nested in a synthetic child node of type
"category". The frontend skips chipletPath for category nodes so they
do not participate in chiplet visibility sync.

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
@maliberty maliberty self-assigned this Jun 9, 2026
@github-actions github-actions Bot added the size/S label Jun 9, 2026

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request introduces a "Backside" category node in the layer hierarchy to group backside layers, and updates the frontend display controls to prevent category nodes from participating in chiplet synchronization. It also adds unit tests to verify this behavior. The reviewer suggests adding a unique "path" field to the synthetic "Backside" category node to maintain API schema consistency and prevent potential parsing issues in clients.

Comment thread src/web/src/tile_generator.cpp
@maliberty maliberty marked this pull request as ready for review June 9, 2026 21:05
@maliberty maliberty requested a review from a team as a code owner June 9, 2026 21:05
@maliberty maliberty requested a review from gadfort June 9, 2026 21:05
@maliberty maliberty merged commit b4d7822 into The-OpenROAD-Project:master Jun 9, 2026
17 checks passed
@maliberty maliberty deleted the web-backside-layer-category branch June 9, 2026 21:21
@maliberty

Copy link
Copy Markdown
Member Author

@mguthaus FYI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants