kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #30888
Re: [PATCH] GerbView GAL support
-- Apologies if this message is a repeat. Looks like first attempt didn't
go through --
Hi Jon-
This is really neat work! I especially like the selection highlighting.
However, I do have a show-stopping (for me) issue. Loading a few of my
projects takes a really long time. Using the legacy canvas, I can load the
attached layer in about 2 seconds on my relatively modest machine. When I
use the OpenGL canvas, it takes almost a full minute. Same again when I
zoom and scroll. There is no lag on the legacy canvas, but at least 2-3
seconds to zoom or scroll under OpenGL.
I am running Linux, latest NVidia drivers on a GeForce 750 GTX.
Best-
Seth
On Thu, Sep 21, 2017 at 3: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
> >>>>
> >>>>
> >>>>
> >>>> _______________________________________________
> >>>> 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
> >>>>
> >>>
> >>>
> >>>
> >>> _______________________________________________
> >>> 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
> >>>
> >>>
> >>
> >
>
>
>
> _______________________________________________
> 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
>
>
Attachment:
8CH_Amplifier-F.Cu.gtl.zip
Description: Zip archive
Follow ups