[bt-devel] Search Analysis

[Martin Gruner]

Hi Gary,

thank you so much for your work! It looks good.

I noticed that when I'm moving the mouse over the analysis, one CPU is being 
used at 100%. I guess that there are too many events being fired of which all 
result in some calculation being done. Can you try to improve that, maybe with 
some QTimers which make sure that updates are only performed once in a while?

mg

On Saturday 29 November 2008 02:50:33 Gary Holmlund wrote:
> Eeli Kaikkonen wrote:
> > Gary Holmlund wrote:
> >> The Search Analysis feature has been turned back on. This had been
> >> turned off during the Qt3 to Qt4 port. I had to fix several issues that
> >> were not done quite right during the port.
> >>
> >> This feature is available from a button in the Search dialog. Any
> >> feedback is welcome.
> >>
> >> Gary
> >
> > Great! It looks good. With quick test I found one bug: the tooltips over
> > the result graph bars show all 0(0%). I don't remember if the
> > calculation algorithm is there - if it's disappeared, you have to find
> > it in 1.6 code.
> >
> > A minor wishlist item is the initial size of the dialog: it's now
> > fullscreen. It's easy to resize after opening but it's a bit annoying
> > because it happens every time. This can be left for later, though. Maybe
> > nobody ever thinks about it because it's a rarely used feature.
> >
> > --Eeli Kaikkonen
> >
> > _______________________________________________
> > bt-devel mailing list
> > bt-devel at crosswire.org
> > http://www.crosswire.org/mailman/listinfo/bt-devel
>
> The tooltip calculation was correct but it was done before the data
> had been setup. I just had to move the place it was called.
>
> I setup the dialog to be non-maximized, 400 pixel high, and the width
> is automatically sized to the data such that the horizontal scrollbar
> is not used, but not larger that the current display screen width.
>
> Gary
>
> _______________________________________________
> bt-devel mailing list
> bt-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/bt-devel