kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@xxxxxxxxx>]

On 5/30/19 4:53 PM, Seth Hillbrand wrote:
> On 2019-05-30 15:00, jp charras wrote:
>> Le 29/05/2019 à 21:31, Seth Hillbrand a écrit :
>>> On 2019-05-29 10:33, jp charras wrote:
>>>> Attached a patch that modify the way filled areas (solid polygons) are
>>>> built in copper areas.
>>>>
>>>> Currently, solid polygons are slightly smaller than the exact area, and
>>>> the polygon outlines have a thickness to fill the exact area.
>>>> With this patch, polygon outlines have no thickness and the polygons
>>>> have the exact area.
>>>>
>>>> To test it on a given zone, the zone setting must be edited with the
>>>> "Fill polys without thick outline" checked.
>>>
>>>
>>> Hi JP-
>>>
>>> Why did you decide to make this a user option?  Is there some feature
>>> that it prevents that a user would want for some areas but not for
>>> others?
>>>
>>> I tested it with a large board and it reduces the polygon point count by
>>> almost 50% (!) for complex fills.  If I zoom in on an edge, it appears
>>> that the approximation count is substantially coarsened by the patch. 
>>> See attached image.  The edge on the right is with the new option
>>> enabled.  The edge of the left is without the new option.
>>>
>>> I didn't find any other issues.  Large boards were much faster and DRC /
>>> plotting appear consistent between options (with the exception noted
>>> above)
>>>
>>> -Seth
>>
>>
>> Thanks Seth for your test.
>>
>> Currently, having a user option is useful to test and compare the 2
>> options (the current way, and the new way).
> 
> OK.  Makes sense.  Instead of changing the board file, can we put the
> option in the advanced config file to enable our testing?  It would be
> nice to avoid changing the file format here.

I agree that the board file format should not change for rendering
configuration.  Please make this a user option.

> 
> If it is useful, I am attaching a patch to do this.  It also uses the
> board settings for approximations to avoid the excess coarsening.
> 
> Feel free to squash this into your patch to minimize the commit size if
> you find it useful.
> 
> Best-
> Seth
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>