← Back to team overview

kicad-developers team mailing list archive

Re: 6.0 Zone filling differences

 

Hi Eeli,

Fixes in.  (I built my own test case so I can’t be 100% sure it will fix yours, but I’d be 90% sure I got it.)

Cheers,
Jeff.


> On 25 Jun 2019, at 23:44, Eeli Kaikkonen <eeli.kaikkonen@xxxxxxxxx> wrote:
> 
> Actually I didn't notice the hole offset thing in the lower oblong pad. I just saw that the upper one doesn't have spokes at all. It's unexplainable when the lower one, identical, does have one spoke (the 45 deg spoke to the small SMD pad is actually a track). And the upper one doesn't have spokes on the bottom layer either while there are two spokes there with the old 5.0.2 version.
> 
> Eeli Kaikkonen
> 
> 
> ke 26. kesäk. 2019 klo 1.08 Jeff Young (jeff@xxxxxxxxx <mailto:jeff@xxxxxxxxx>) kirjoitti:
> Great test-case: an offset and rotated pad.
> 
> So the old algorithm centred the spokes on the pad; the new one centres them on the hole.  I assume old is preferred?
> 
> Cheers,
> Jeff.
> 
> 
>> On 25 Jun 2019, at 22:44, Eeli Kaikkonen <eeli.kaikkonen@xxxxxxxxx <mailto:eeli.kaikkonen@xxxxxxxxx>> wrote:
>> 
>> There's something wrong in creating the thermal for the oblong pad in the attached picture. Left side: 5.0.2. Right: includes your commit. Tested on Linux.
>> 
>> Here's also the footprint, taken from the board file:
>> 
>> (module "XX-X-USB:Microusb_female_thrhole_horn" (layer "F.Cu") (tedit 5C1B9BE7) (tstamp 5D08B5B8)
>>     (at 119.25 105.8 90)
>>     (descr "MICRO USB R/A-473460001")
>>     (path "/5CDA4205")
>>     (attr smd)
>>     (fp_text reference "X2" (at 0.15 -6.15 90) (layer "F.SilkS")
>>       (effects (font (size 1 1) (thickness 0.15)))
>>     )
>>     (fp_text value "USB_B_Micro" (at -0.2 4.2 90) (layer "F.SilkS")
>>       (effects (font (size 1 1) (thickness 0.15)))
>>     )
>>     (fp_line (start -3.75 2.75) (end -3.75 -2.75) (layer "F.Fab") (width 0.127))
>>     (fp_line (start 3.75 2.75) (end -3.75 2.75) (layer "F.Fab") (width 0.127))
>>     (fp_line (start 3.75 -2.75) (end 3.75 2.75) (layer "F.Fab") (width 0.127))
>>     (fp_line (start -3.75 -2.75) (end 3.75 -2.75) (layer "F.Fab") (width 0.127))
>>     (fp_circle (center -1.7 -3.8) (end -1.6 -3.8) (layer "F.SilkS") (width 0.15))
>>     (fp_line (start 4.25 2.65) (end -4.25 2.65) (layer "F.CrtYd") (width 0.05))
>>     (fp_line (start 4.25 -3.75) (end 4.25 2.65) (layer "F.CrtYd") (width 0.05))
>>     (fp_line (start -4.25 -3.75) (end 4.25 -3.75) (layer "F.CrtYd") (width 0.05))
>>     (fp_line (start -4.25 2.65) (end -4.25 -3.75) (layer "F.CrtYd") (width 0.05))
>>     (fp_line (start 3.9 -2.95) (end 3.65 -2.95) (layer "F.SilkS") (width 0.127))
>>     (fp_line (start -3.9 -2.95) (end -3.65 -2.95) (layer "F.SilkS") (width 0.127))
>>     (fp_line (start 3.9 -2.95) (end 3.9 -1.35) (layer "F.SilkS") (width 0.127))
>>     (fp_line (start -3.9 -2.95) (end -3.9 -1.35) (layer "F.SilkS") (width 0.127))
>>     (fp_line (start -5.45 1.35) (end 5.5 1.35) (layer "F.Fab") (width 0.127))
>>     (fp_text user "%V" (at -0.15 4.15 90) (layer "F.Fab")
>>       (effects (font (size 1 1) (thickness 0.15)))
>>     )
>>     (pad "6" thru_hole oval (at -3.575 0 90) (size 1 2) (drill oval 0.6 1.2 (offset 0 -0.15)) (layers *.Cu *.Mask "F.Paste")
>>       (net 2 "GND") (solder_paste_margin -0.01) (solder_paste_margin_ratio -0.0001))
>>     (pad "6" thru_hole oval (at 3.575 0 90) (size 1 2) (drill oval 0.6 1.2 (offset 0 -0.15)) (layers *.Cu *.Mask "F.Paste")
>>       (net 2 "GND") (solder_paste_margin -0.01) (solder_paste_margin_ratio -0.0001))
>>     (pad "6" thru_hole circle (at -2.425 -2.73 90) (size 1 1) (drill 0.6) (layers *.Cu *.Mask "F.Paste")
>>       (net 2 "GND") (solder_paste_margin -0.01) (solder_paste_margin_ratio -0.0001))
>>     (pad "6" thru_hole circle (at 2.425 -2.73 90) (size 1 1) (drill 0.6) (layers *.Cu *.Mask "F.Paste")
>>       (net 2 "GND") (solder_paste_margin -0.01) (solder_paste_margin_ratio -0.0001))
>>     (pad "1" smd roundrect (at -1.3 -2.66 90) (size 0.4 1.5) (layers "F.Cu" "F.Paste" "F.Mask") (roundrect_rratio 0.25)
>>       (net 34 "<root sheet>VBUS") (clearance 0.19))
>>     (pad "2" smd roundrect (at -0.65 -2.66 90) (size 0.4 1.5) (layers "F.Cu" "F.Paste" "F.Mask") (roundrect_rratio 0.25)
>>       (net 62 "<root sheet>D-"))
>>     (pad "3" smd roundrect (at 0 -2.66 90) (size 0.4 1.5) (layers "F.Cu" "F.Paste" "F.Mask") (roundrect_rratio 0.25)
>>       (net 63 "<root sheet>D+"))
>>     (pad "4" smd roundrect (at 0.65 -2.66 90) (size 0.4 1.5) (layers "F.Cu" "F.Paste" "F.Mask") (roundrect_rratio 0.25)
>>       (net 59 "<root sheet>id_r"))
>>     (pad "5" smd roundrect (at 1.3 -2.66 90) (size 0.4 1.5) (layers "F.Cu" "F.Paste" "F.Mask") (roundrect_rratio 0.25)
>>       (net 2 "GND") (clearance 0.19))
>>     (model "${KISYS3DMOD}/Connector_USB.3dshapes/USB_Micro-B_Molex_47346-0001.wrl"
>>       (offset (xyz 0 1.2 0))
>>       (scale (xyz 1 1 1))
>>       (rotate (xyz 0 0 0))
>>     )
>>   )
>> 
>> 
>> __________________
>> BTW, highlighting items in pcbnew seems to be broken, maybe because of the new "real-time highlighting" for delete tool. The delete tool highlight doesn't highlight tracks. The older selection clarification and DCR dialog item highlighting don't work well, only some things are highlighted sometimes. This happens with
>> 
>> Application: Pcbnew
>> Version: (5.1.0-1126-g107d206db), release build
>> Libraries:
>>     wxWidgets 3.0.4
>> Platform: Linux 4.15.0-51-generic x86_64, 64 bit, Little endian, wxGTK
>> Build Info:
>>     wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 3.22
>>     Boost: 1.65.1
>>     OpenCASCADE Community Edition: 6.9.1
>>     Compiler: GCC 7.4.0 with C++ ABI 1011
>> 
>> Build settings:
>>     KICAD_SCRIPTING=ON
>>     KICAD_SCRIPTING_MODULES=ON
>>     KICAD_SCRIPTING_PYTHON3=ON
>>     KICAD_SCRIPTING_WXPYTHON=ON
>>     KICAD_SCRIPTING_WXPYTHON_PHOENIX=ON
>>     KICAD_SCRIPTING_ACTION_MENU=ON
>>     BUILD_GITHUB_PLUGIN=OFF
>>     KICAD_USE_OCE=ON
>>     KICAD_USE_OCC=OFF
>>     KICAD_SPICE=OFF
>> 
>> __________________
>> 
>> ti 25. kesäk. 2019 klo 23.12 Jeff Young (jeff@xxxxxxxxx <mailto:jeff@xxxxxxxxx>) kirjoitti:
>> Whoo hooo!
>> 
>> An algorithm that doesn’t cut any conceptual corners (and so should be correct), and is fast as well.
>> 
>> As always, please send in any exceptions.
>> 
>> Cheers,
>> Jeff.
>> <Screenshot_left_502_right_g107d206db.png>_______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>> Unsubscribe : https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
> 


Follow ups

References