kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #36047
Re: [PATCH] Large board speed
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Suminski <maciej.suminski@xxxxxxx>
-
Date:
Sun, 3 Jun 2018 22:44:28 +0200
-
Authentication-results:
spf=pass (sender IP is 188.184.36.48) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<e088b196-5a1d-b885-46e4-ae70b4715495@gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0
FWIW: I have routed a simple board (166 track segments) with the patch
applied, no issues found.
Cheers,
Orson
On 06/01/2018 11:17 PM, Wayne Stambaugh wrote:
> On 6/1/2018 3:58 PM, Tomasz Wlostowski wrote:
>> On 01/06/18 20:23, Seth Hillbrand wrote:
>>> I think maybe we are seeing the severity of these bugs differently.
>>> There are three substantial issues addressed by this patch. While they
>>> don't affect many users, they do prevent KiCad from being used for
>>> larger, more complex boards that we nominally support. This is a also
>>> regression from v4.
>>
>> Hi all,
>>
>> I vote for merging Seth's patch.
>>
>> For those concerned about its correctness: the connectivity algo is
>> designed in such way that in case of bugs in collision search (other
>> than in BOARD_ITEM::HitTest()/POLY_GRID_PARTITION which haven't been
>> touched for a year), it may find false unconnected items, but not false
>> connected ones. In the worst case, we'll have false 'unconnected items'
>> warnings in the DRC, but I doubt there will be any. The patch is simple
>> and elegant.
>>
>> Tom
>
> Tom,
>
> Have you actually tested the patch? Voting for it doesn't help me make
> an informed decision. What I really need is test feedback. I'm not
> opposed to merging it but I would like as large of a test group as
> possible so I have some confidence that it's not going to cause a bunch
> of issues this close to the stable release. I plan on testing it
> tomorrow but I am an infinite sample of one. A few other testers would
> be appreciated before I'm willing to sign off on committing it.
>
> Cheers,
>
> Wayne
>
>
> _______________________________________________
> 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
>
Follow ups
References
-
[PATCH] Large board speed
From: Seth Hillbrand, 2018-05-25
-
Re: [PATCH] Large board speed
From: Tomasz Wlostowski, 2018-05-25
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-05-25
-
Re: [PATCH] Large board speed
From: Tomasz Wlostowski, 2018-05-28
-
Re: [PATCH] Large board speed
From: Wayne Stambaugh, 2018-05-29
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-05-29
-
Re: [PATCH] Large board speed
From: Jeff Young, 2018-05-29
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-05-29
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-05-29
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-06-01
-
Re: [PATCH] Large board speed
From: Simon Richter, 2018-06-01
-
Re: [PATCH] Large board speed
From: Seth Hillbrand, 2018-06-01
-
Re: [PATCH] Large board speed
From: Tomasz Wlostowski, 2018-06-01
-
Re: [PATCH] Large board speed
From: Wayne Stambaugh, 2018-06-01