Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for circles and arcs in symbols #45

Closed
Bouni opened this issue Jun 9, 2022 · 13 comments
Closed

Add support for circles and arcs in symbols #45

Bouni opened this issue Jun 9, 2022 · 13 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@Bouni
Copy link
Contributor

Bouni commented Jun 9, 2022

By now circles and arcs are not supported which leads to wierd symbles sometimes, here's nan example for C318884

Easyeda symbol:
image

KiCAD symbol generated by this tool:
image

According to the code, circles and arcs are not yet implemented:

# ---------------- CIRCLE ----------------
class EeSymbolCircle(BaseModel):
... # TODO
# ---------------- ARC ----------------
class EeSymbolArc(BaseModel):
... # TODO

The code even states that arcs are not supported by KiCAD but thats not true for V6.0 / V6.99 (elliptical arcs are not supported, thats true)

elif designator == "PI":
# Elliptical arc seems to be not supported in Kicad
...
# For ellipse
elif designator == "E":
... # Ellipse seems to be not supported in Kicad
# For arcs
elif designator == "A":
... # TODO

@uPesy uPesy added enhancement New feature or request bug Something isn't working labels Jun 9, 2022
@Bouni
Copy link
Contributor Author

Bouni commented Jun 13, 2022

Nice, circles seem to work in the latest dev commit 😍

grafik

grafik

@uPesy
Copy link
Owner

uPesy commented Jun 13, 2022

Yes but i'm not sure if i should enable filling the background (having the circle with yellow background)
The circle are descrided as ellipse in the easyeda footprint 🤔

@Bouni
Copy link
Contributor Author

Bouni commented Jun 14, 2022

Does the easyeda data show if their symbol has filled circles or not?

@uPesy
Copy link
Owner

uPesy commented Jun 14, 2022

No i don't think so

@Bouni
Copy link
Contributor Author

Bouni commented Jun 14, 2022

In that case I would no fill them in ...

@uPesy
Copy link
Owner

uPesy commented Jun 15, 2022

Yeah you're right, but in easyeda, there is no background filling on symbol, contrary to Kicad :
image
image

@Bouni
Copy link
Contributor Author

Bouni commented Jun 15, 2022

For C318884: background fill in circles is gone, looks good to me!

image

The optocoupler issue is answered in #53

@uPesy
Copy link
Owner

uPesy commented Jun 19, 2022

Symbol arc is now supported in the last version v0.5.0.
Note that easyeda use elleptical arc while Kicad support only circle arc.

@Bouni
Copy link
Contributor Author

Bouni commented Jun 20, 2022

Are you aware of a symbol that use actual elliptical arcs?

I mean they most likely use elliptical arcs as "normal" arcs but never as an elliptical shape, right?

@uPesy
Copy link
Owner

uPesy commented Jun 20, 2022

For the coil, the radius is rx=3.9 and ry=4.0 on Easyeda

I mean they most likely use elliptical arcs as "normal" arcs but never as an elliptical shape, right?
Yes

@Bouni
Copy link
Contributor Author

Bouni commented Jun 20, 2022

What do you plan to do in such cases?
I could imagine having a threshold for the difference in rx and ry of lets say 0.1mm and if that threshold is not exceeded use one of the values as arc radius. That way we end up wit usable symbols that might not fit a 100% but still better than skip the arcs completely.

@uPesy
Copy link
Owner

uPesy commented Jun 20, 2022

For now it takes the maximum value from rx and ry

@Bouni
Copy link
Contributor Author

Bouni commented Jun 20, 2022

Maybe easier as my suggestion. Thinks that'll do the job.

@Bouni Bouni closed this as completed Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants