wokwi / easyeda2kicad

Convert EasyEDA designs to KiCad EDA
https://wokwi.com/tools/easyeda2kicad
Other
287 stars 39 forks source link

Non-GND solid regions are missing after conversion #17

Closed mtiutiu closed 4 years ago

mtiutiu commented 4 years ago

Description: If I have in the original design a solid copper region created say for a MOSFET cooling and/or for high current paths which are not tied to GND then those aren't being translated.

Expected result: The solid region tied to MOSFET drain pad should be present on both sides of the PCB.

See attached pictures.

Original: original

Another two pictures (top/bottom) for reference: Screenshot from 2020-01-30 10-01-03 Screenshot from 2020-01-30 10-00-45

Conversion result: Screenshot from 2020-01-30 09-49-46

Project files: project.zip

KiCAD version (latest stable): 5.1.5 easyeda2kicad version: 1.3.4

If any other details are needed please let me know. Thanks.

urish commented 4 years ago

Could be related to #15

mtiutiu commented 4 years ago

@urish #15 seems to be fixed now and the symbols are rendered correctly. The solid region is still missing though.

urish commented 4 years ago

Right, I'll look into this over the next few days

urish commented 4 years ago

How about now?

mtiutiu commented 4 years ago

Yes sir - that's the spirit! Works like magic now. Thanks a lot.

One small issue is present but that's fixable by hand. It seems that the vias inside that solid region even if they have associated the same net name still don't want to connect.

Other than that - respect!!!

Screenshot from 2020-02-04 12-24-19

Vias appear like not being connected: Screenshot from 2020-02-04 12-27-32

urish commented 4 years ago

Thanks for the super-quick feedback!

Yes, I was looking in KiCad, but it seems like they do not support setting the net names for polygons (I even tried adding a net attribute, but it failed so I commented that out). If you do find a way to do this via KiCad UI, let me know and I will look how to implement it.

mtiutiu commented 4 years ago

@urish I think this ticket can be closed as the initial issue was solved. The rest..well maybe other tickets if it's the case.

mtiutiu commented 4 years ago

@urish can we have a new release including this fix also if not already there? Thanks.

urish commented 4 years ago

Sure, released as 0.4.1