Skip to content

Add PMP11282 schematic rendering reproduction - #264

Open
ShiboSoftwareDev wants to merge 1 commit into
mainfrom
codex/repro-pmp11282-schematic-boxes
Open

Add PMP11282 schematic rendering reproduction#264
ShiboSoftwareDev wants to merge 1 commit into
mainfrom
codex/repro-pmp11282-schematic-boxes

Conversation

@ShiboSoftwareDev

@ShiboSoftwareDev ShiboSoftwareDev commented Aug 25, 2026

Copy link
Copy Markdown
Contributor
Screenshot 2026-08-25 at 11 29 34 PM

Summary

  • add the compiled Circuit JSON for the source-derived PMP11282 isolated DC/DC sheet
  • add a React Cosmos fixture that renders the complete 112-component schematic
  • preserve the current 160 routed traces and 73 fallback/net labels for visual inspection
  • reproduce the generic yellow component bodies around the transformer, coupled inductor, optocouplers, and dual diodes

This is reproduction-only. It does not change viewer or SVG production behavior.

Ownership finding

SchematicViewer delegates schematic SVG creation to circuit-to-svg. In this fixture the affected Circuit JSON components have symbol_name: null, so the viewer has no domain symbol to render and correctly falls back to generic component bodies. T500 also arrives with an explicitly undersized 0.4 mm body.

The fixture therefore establishes the visual case without claiming the viewer is the root cause. Follow-up work should separately address:

  • missing domain symbols in core / schematic-symbols
  • removal of undersized generic-body dimensions in the TI TSX
  • renderer behavior only if a valid symbol/body still renders incorrectly

Source

Validation

  • bun test
  • bun run build
  • bunx tsc --noEmit
  • bun run format:check
  • visually verified in the new Cosmos fixture

@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
schematic-viewer Ready Ready Preview Aug 25, 2026 8:24pm

Request Review

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

what are you repro'ing?

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