wokwi / easyeda2kicad

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

Some silkscreen symbols are not being rendered completely #15

Closed mtiutiu closed 4 years ago

mtiutiu commented 4 years ago

Description: Seems that some symbols present on the PCB silkscreen (top and/or bottom) are not converted completely leading to an incomplete drawing. See the "high voltage symbols" in the attached pictures.

Expected result: Any silkscreen symbols should be fully translated.

Original: Screenshot from 2020-01-29 15-34-47

Conversion result: Screenshot from 2020-01-30 09-30-27

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

Can you please check again with latest version from master?

mtiutiu commented 4 years ago

@urish

Hi, the conversion process is crashing with an exception now:

/home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:319
    var isSmd = shapes.some(function (shape) { return shape[0] === 'pad' && shape[2] === 'smd'; });
                                                           ^

TypeError: Cannot read property '0' of null
    at /home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:319:60
    at Array.some (<anonymous>)
    at convertLib (/home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:319:24)
    at convertShape (/home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:391:21)
    at /home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:403:148
    at Array.map (<anonymous>)
    at Object.convertBoard (/home/mtiutiu/Tmp/easyeda2kicad/dist/board.js:403:118)
    at Object.<anonymous> (/home/mtiutiu/Tmp/easyeda2kicad/dist/main.js:17:84)
urish commented 4 years ago

Should be fixed by #20, can you give it another go?

mtiutiu commented 4 years ago

Yep. Works now as expected. See attached picture. Screenshot from 2020-02-04 10-57-32

urish commented 4 years ago

Awesome! released as 1.4.0