# Techmannih - @Seve there are some concave and s...

Channel: #contributor
Source: https://discord.com/channels/1233487248129921135/1459046553934237740
Started: 2026-01-09T04:50:06.905000+00:00
Last activity: 2026-02-27T03:14:42.349000+00:00

## techmannih — 2026-01-09T04:50:06.232000+00:00

<@757706909351411845> there are some concave and some convex. can we use them with flags convex or concave?
Attachment: Screenshot_2026-01-09_10-18-34.png — https://community.tscircuit.com/media/1459046553875648634

## Seve — 2026-01-09T04:50:23.433000+00:00

yes we should add those flags

## techmannih — 2026-01-09T04:51:21.191000+00:00

is there any default use like 0402x2 which is for default convex or concave

## Seve — 2026-01-09T04:51:49.804000+00:00

probably convex is the default

## techmannih — 2026-01-11T11:35:15.426000+00:00

Sorry I am founding lot of unwanted thing in 3d-viewer when i try to refactor with circuit-to-canvas.

## techmannih — 2026-02-09T03:43:16.621000+00:00

I think KiCad is using clockwise pin numbering. The green color is from KiCad, and the red one is ours.
Attachment: Screenshot_2026-01-30_14-17-18.png — https://community.tscircuit.com/media/1470263760554954875

## Seve — 2026-02-09T03:44:54.810000+00:00

Hmm it seems like just a bug with the conversion not related to clockwise/ccw

## Seve — 2026-02-09T03:45:13.631000+00:00

Because if we’re converting custom symbols it should just adopt whatever we say

## techmannih — 2026-02-09T03:46:46.534000+00:00

yes for custom symbols, the conversion should follow exactly what we define

## techmannih — 2026-02-26T03:00:52.781000+00:00

<@757706909351411845> yep, there is the rotation issue, we doest support the rotation to the polygon smtpad. do we need to add support rotation. ???
Attachment: Screenshot_2026-02-26_08-27-21.png — https://community.tscircuit.com/media/1476413683985813578

## Seve — 2026-02-26T03:01:36.464000+00:00

i don't know what we need to do exactly, but polygons are never represented with rotations in circuit json

## Seve — 2026-02-26T03:02:06.639000+00:00

it's not tricky

## techmannih — 2026-02-26T03:18:52.615000+00:00

yep its fixed, i was thinking in different way. 
ty!
Attachment: Screenshot_2026-02-26_08-48-04.png — https://community.tscircuit.com/media/1476418212852797684

## techmannih — 2026-02-26T03:39:18.441000+00:00

https://github.com/tscircuit/kicad-to-circuit-json/pull/36

## techmannih — 2026-02-26T05:42:39.318000+00:00

Removing incorrect logic handling that was affecting the pill-shaped plated hole. 
https://github.com/tscircuit/kicad-to-circuit-json/pull/38

## techmannih — 2026-02-26T05:45:48.338000+00:00

Yes, I have added rotation support for circular holes and rectangular pads. It is also being generated in the circuit JSON, but I still need to figure out what the issue is in this case and why it is not working. I am also looking on it.
Attachment: Screenshot_2026-02-26_11-14-49.png — https://community.tscircuit.com/media/1476455189056721016

## Seve — 2026-02-26T05:46:18.981000+00:00

sg!

## techmannih — 2026-02-26T06:26:21.395000+00:00

https://github.com/tscircuit/circuit-to-canvas/pull/184

## techmannih — 2026-02-26T18:10:24.640000+00:00

I’ve fully switched to using the transformation matrix for the plated hole. All ctx.translate, ctx.save, ctx.restore, and ctx.rotate calls have been removed. I’ll address the rotation handling separately later on. Could you please review?
https://github.com/tscircuit/circuit-to-canvas/pull/189

## Seve — 2026-02-26T18:46:11.446000+00:00

i'm not really sure what to do because i've given a lot of feedback to PRs but the quality doesn't seem to be getting better, i just don't have time to review PRs with a lot of mistakes

## Seve — 2026-02-26T18:49:14.381000+00:00

you have single letter functions and variables like `mat`

## Seve — 2026-02-26T18:50:40.585000+00:00

you also used the word `rotation` for a fourth time

## techmannih — 2026-02-27T02:16:11.618000+00:00

these are matrix and radian, yeah i change them.

## techmannih — 2026-02-27T02:16:28.225000+00:00

No, it was using ccwRotationDegrees.

## techmannih — 2026-02-27T02:17:08.295000+00:00

[attachment]
Attachment: Screenshot_2026-02-27_07-46-53.png — https://community.tscircuit.com/media/1476765064030716006

## Seve — 2026-02-27T02:19:01.760000+00:00

[attachment]
Attachment: image0.jpg — https://community.tscircuit.com/media/1476765539878830080

## techmannih — 2026-02-27T02:20:35.450000+00:00

oh variable name, it was param

## techmannih — 2026-02-27T02:38:15.989000+00:00

Can you reopen it? I’ve applied the changes.

## Seve — 2026-02-27T03:04:38.656000+00:00

you need to get another reviewer before requesting my reviews, you need to get two reviews now for your PRs

## Seve — 2026-02-27T03:05:04.384000+00:00

the reviews must come from a maintainer

## techmannih — 2026-02-27T03:14:42.349000+00:00

sure