← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Show the busy cursor while loading libraries

 

On Thu, Mar 09, 2017 at 09:46:54AM +1100, Oliver Walters wrote:
> >
> > Slow?  I have noticed that the library loading slowness appeared with
> > the new IO manager thing. Although I don't have any proff to quanify
> > that.
> 
> 
> I can also attest that (at least on Windows) the more recently nightlies
> (corresponding to the new IO plugin, approximately) take ~10s to load a
> library set that was previously < 1s.
> 
> Once the libraries are loaded, eeschema takes an additional length of time
> (~10s) to open, whereas previously it opened "instantly"

I've noticed this too. Funny enough it doesn't seem to happen for all
projects for me. I wonder if it's something only being done sometimes,
or a polynomial time explosion.

Either way someone needs to point the business end of a profiler at the
library loader.

> 
> On Thu, Mar 9, 2017 at 8:54 AM, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:
> 
> > 2017-03-08 22:33 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
> > > On 3/8/2017 4:08 PM, Chris Pavlina wrote:
> > >> That's why I submitted such a trivial patch to the list first, I figured
> > >> someone would say something ;)
> > >>
> > >> It does make sense to me, because the GUI is blocked. The busy cursor
> > >> says to me "yes, the GUI is supposed to be blocked right now, it's not
> > >> frozen". Even with a progress bar, it can seem unresponsive -
> > >> particularly if 1) the progress bar ends up obscured, as can happen with
> > >> 'weird' window managers sometimes, or 2) if a single library takes a
> > >> particularly long time to load, which I'm sure will only get worse if we
> > >> eventually allow loading them over the internet like for footprint libs.
> > >
> > > It's the old belt and suspender method.  Users have got to quit using
> > > those 'weird' window managers.  It causes way too much grief.  I'm
> > > surprised that a single library load takes long enough to need a busy
> > > cursor but I'm not opposed to the patch.  Does the progress bar in
> > > wxWidgets have a continuous mode?  That doesn't solve the hidden
> > > progress window issue though.
> > >
> >
> > I like the non-continous mode.
> >
> > Slow?  I have noticed that the library loading slowness appeared with
> > the new IO manager thing. Although I don't have any proff to quanify
> > that.
> >
> > >>
> > >>
> > >> On Wed, Mar 08, 2017 at 03:55:10PM -0500, Wayne Stambaugh wrote:
> > >>> Does showing a progress dialog and a busy cursor at the same time make
> > >>> sense?
> > >>>
> > >>> On 3/8/2017 3:36 PM, Chris Pavlina wrote:
> > >>>> Hi,
> > >>>>
> > >>>> This patch enables display of the "busy" cursor while schematic
> > >>>> libraries are being loaded. Tested on Linux, Windows 10, and macOS
> > >>>> 10.12.
> > >>>>
> > >>>>
> > >>>>
> > >>>> _______________________________________________
> > >>>> 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
> >
> > _______________________________________________
> > 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