← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] GerbView GAL support

 

Le 21/09/2017 à 14:04, Wayne Stambaugh a écrit :
> Orson,
> 
> I'm leaving this decision up to you and JP since I have not had time to
> test it.  JP, do you have any other concerns about this?

Sorry for the delay, but I was away last 3 days.
I do not have concerns about merging Gerbview GAL.

We know there are a few issues, but they can be fixed later. None is blocking.
And the legacy canvas can be used with no issue.

So, Orson, if you can commit this very good enhancement, please do it.
AFAIK, remember also only the GAL canvas works on wxWidgets + GTK3, so the GAL canvas support is
very important for the future of KiCad.

Thanks to Jon and Orson for all this work.

> 
> Thanks,
> 
> Wayne
> 
> On 9/21/2017 6:05 AM, Maciej Sumiński wrote:
>> Hi Jon,
>>
>> Thanks you, this is really cool! Now it is even more tempting to merge
>> the gerbview_gal branch. I am going to wait one more day for vetos and
>> tomorrow I will push it to the master branch.
>>
>> Regards,
>> Orson
>>
>> On 09/20/2017 09:57 PM, Jon Evans wrote:
>>> Hi Orson,
>>>
>>> Give this a shot in your branch.  It should work in pcbnew also now.
>>>
>>> -Jon
>>>
>>> On Wed, Sep 20, 2017 at 9:28 AM, Jon Evans <jon@xxxxxxxxxxxxx> wrote:
>>>
>>>> Hi Orson,
>>>>
>>>> Thank you for staging this for merge on your branch.  I checked and you do
>>>> have all the patches.
>>>>
>>>> 1) Yes I planned on refactoring the selection tool once things stabilized
>>>> with the highlighting etc.
>>>> 2) Do you mean when you are highlighting Gerber X2 attributes, or when you
>>>> are deselecting things, or something else?
>>>> 3) That's a good idea on VIEW_GROUP, I will give it a try and send a patch.
>>>>
>>>> Thanks,
>>>> Jon
>>>>
>>>> On Wed, Sep 20, 2017 at 5:46 AM, Maciej Sumiński <maciej.suminski@xxxxxxx>
>>>> wrote:
>>>>
>>>>> Hi Jon,
>>>>>
>>>>> GALifying GerbView is a huge task, so thank you very much for your work!
>>>>> I have just tested your changes and in my opinion it is in a state that
>>>>> deserves merging and further tests. The new way of item highlighting is
>>>>> awesome, we need to port it to pcbnew as well.
>>>>>
>>>>> For now I keep your patches in a separate branch, with some minor
>>>>> modifications on top of it [1]. Please verify it contains all the needed
>>>>> patches. If nobody objects, I would like to merge it this week.
>>>>>
>>>>> Just a few minor remarks:
>>>>> - It seems there is some code that could be refactored to share it with
>>>>> pcbnew (e.g. selection tool).
>>>>> - 'Clear highlight' operation takes long time to finish (seems more than
>>>>> with the legacy canvas), but I cannot really see what is happening
>>>>> there. If it cannot be easily fixed, perhaps it could set the mouse
>>>>> cursor to busy.
>>>>> - For the new highlighting method: perhaps a more universal way is to
>>>>> create a temporary VIEW_GROUP object containing the selection candidate.
>>>>> This way it can be temporarily displayed on the overlay layer, without
>>>>> modifying the original ViewGetLayer() methods.
>>>>>
>>>>> Regards,
>>>>> Orson
>>>>>
>>>>> 1. https://code.launchpad.net/~orsonmmz/kicad/+git/kicad/+ref/
>>>>> gerbview_gal
>>>>>
>>>>> On 09/18/2017 12:47 AM, Jon Evans wrote:
>>>>>> Hi all,
>>>>>>
>>>>>> The day has finally come!  I have distilled my GerbView GAL branch into
>>>>> a
>>>>>> patchset attached to this email.  Hopefully with this merged into
>>>>> master we
>>>>>> can identify any remaining bugs and clean it up for 5.0.
>>>>>>
>>>>>> Note that this set is split into 5 patches to make review easier, but
>>>>> they
>>>>>> are not intended to compile and work independently.
>>>>>>
>>>>>> Best,
>>>>>> Jon



-- 
Jean-Pierre CHARRAS


Follow ups

References