Skip to content

fix: preserve EasyEDA schematic arc geometry in generated symbols - #511

Open
anil08607 wants to merge 2 commits into
tscircuit:mainfrom
anil08607:fix/schematic-arc-transform-endpoints
Open

fix: preserve EasyEDA schematic arc geometry in generated symbols#511
anil08607 wants to merge 2 commits into
tscircuit:mainfrom
anil08607:fix/schematic-arc-transform-endpoints

Conversation

@anil08607

Copy link
Copy Markdown
Contributor

Summary

  • Parse complete EasyEDA schematic arc data, including X/Y radii, rotation, large-arc flag, and sweep flag.
  • Convert circular and near-circular SVG endpoint arcs into native <schematicarc> elements.
  • Preserve arc direction, angles, stroke width, and color.
  • Fall back to <schematicpath> for elliptical or unsupported arcs.
  • Update inductor conversion tests and SVG snapshots, including C1046 and C281113.

Result

EasyEDA inductor symbols now render with smooth coil arcs instead of path-based approximations, matching the JLCPCB symbol more closely.

Testing

  • bun test --timeout 20000 — 192 passed, 0 failed
  • bun run build

@anil08607 anil08607 changed the title feat: add native schematic arc support for EasyEDA symbols fix: preserve EasyEDA schematic arc geometry in generated symbols Aug 24, 2026

@imrishabh18 imrishabh18 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't seem like the right place to fix this, the easyeda converter is emitting the valid arc A

Comment on lines -45 to -48
<schematicpath svgPath="M -0.3376 0.0014 A 0.08 0.078 0 1 0 -0.1784 0.0012" strokeColor="#880000" />
<schematicpath svgPath="M -0.168 0.0014 A 0.08 0.078 0 1 0 -0.0088 0.0014" strokeColor="#880000" />
<schematicpath svgPath="M 0.0014 0.0014 A 0.08 0.078 0 1 0 0.1606 0.0014" strokeColor="#880000" />
<schematicpath svgPath="M 0.174 0.0014 A 0.08 0.078 0 1 0 0.3334 0.0012" strokeColor="#880000" />

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't seem like the right place to fix this, the easyeda converter is emitting the valid arc A

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