<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 7, 2015 at 7:10 PM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">win64, dual core cpu<br>
<br>
general performance drops are observed in both official binaries and<br>
my local debug build.<br>
of course we can wait until *actual* users complain, instead of me.<br>
just a report, as i don't have time to fix any of that ATM. perhaps<br>
later this month...<br>
<br>
i guess this is mostly in Tomaz' field.<br>
<br>
with the recent addition of the mean depth line and text, i've noticed<br>
further increase of cpu usage when hovering the profile. so i've<br>
started "profiling the profile" to see what are the main causes for<br>
the high cpu usage 30%.<br>
<br>
i've realized it's a multitude of things and all contribute a<br>
millisecond or two so that it takes a while to render a frame in the<br>
UI thread. to explain the issue i'm describing, imagine the mean depth<br>
line jumping at 500ms intervals instead of smoothly following the<br>
mouse.<br></blockquote><div><br></div><div>Actually, the line doesn't smootly follows the mouse for other reasons,<br>if you see that the tooltipitem is not smootly updating, then there's a real issue.<br><br></div><div>the line doesn't smootly follows the mouse because my implementation for it is wrong:<br></div><div>if the current mouse position is not at a time that exists in the plot_info datastructure, it will not<br></div><div>generate a line for that position.<br><br></div><div>belive me, I'v tested by creating fake information and it worked.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
1) the main hog is ToolTipItem - 20%<br>
perhaps elements can be re-used instead of re-creating them each time<br>
the mouse moves.<br>
also, we can possibly get rid of the animation and store some of the<br>
VALUE / 2 operations in VALUE_D_2 consts. etc...<br>
not sure how well this will work without testing it.<br></blockquote><div><br></div><div>The current API of the ToolTipItem supports this, but the main tooltip used is a *huge* text, sometimes with more than 16 lines of code<br>we need to clean that - urgently.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) those extra 10% are caused by the new InstantMeanDepthLine::mouseMoved()<br>
as it calculates based on the running average each time instead of<br>
using a lookup table.<br></blockquote><div><br></div><div>Hm... can you ( or anyone else ) help me to create a lookup table?<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
lubomir<br>
--<br>
</blockquote></div><br></div></div>