Skip to content

feat: Use sagitta to draw arcs.#789

Merged
eyal0 merged 1 commit into
pcb2gcode:masterfrom
eyal0:sagitta
Mar 8, 2026
Merged

feat: Use sagitta to draw arcs.#789
eyal0 merged 1 commit into
pcb2gcode:masterfrom
eyal0:sagitta

Conversation

@eyal0

@eyal0 eyal0 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

This fixes #786.

@eyal0

eyal0 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

This is failing on runs without geos.

@jubilee4321

Copy link
Copy Markdown

The gerber and the processed output are different

Gerber on the left, processed output on the right. The circle is shifted to the left and overlaps the left edge.
gerbv+pcb2gcode

@jubilee4321

Copy link
Copy Markdown

Nevermind, I just needed a better gerber viewer.
better-gerber

@coveralls

coveralls commented Mar 8, 2026

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 72.2%. remained the same
when pulling 5740f66 on eyal0:sagitta
into c36a1a1 on pcb2gcode:master.

@eyal0
eyal0 merged commit f496334 into pcb2gcode:master Mar 8, 2026
19 checks passed
@eyal0 eyal0 added the feature label Mar 8, 2026
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.

Some nearly-connected lines in the gerber input have been adjusted to properly connect (continuation)

3 participants