[Sugar-devel] [Dextrose] [PATCH v5 sugar] Pulsing icon delayed by 5 seconds or so SL#2080

[Anurag Chowdhury]

On Sun, Oct 31, 2010 at 12:14 PM, Martin Dengler
<martin at martindengler.com> wrote:
> On Sat, Oct 30, 2010 at 12:09:53AM +0530, Anurag Chowdhury wrote:
>> Uptill now what we have gathered from this issue is:
>>
>> 1) We can improve this issue by using a cache system or cairo operation:
>>
>>   >    " I think the problem is the code is rendering the svg icon every time
>>   >      the color is changed.
>>   >      There are not cache or cairo operation used to avoid this.
>>   >
>>   >      Gonzalo"
>>
>>  -- I think that we all seem to agree upon this point that using a
>> cache to store the once rendered frames and then reuse them later
>>    in the animation to improve the performance of the system.
>
> This is already done.
>
>>  3) We can try some tweaks to the animation and related aspects of the
>> issue to make the animation smoother and lighter on
>>     system:
>>
>>   >  "We did play around with a few ideas when Wade last worked on
>> the pulse optimisation/enhancements. He landed
>>   >  improvements for speed by minimising the area being redrawn, and
>> landed the addition of the "failed to launch" check and
>>   >  button/message. He also found a few extra cpu seconds worth of
>> improvement that didn't land in time. The trick was to fade
>>   >  in, _hold_, fade out, _hold_, the effect is still of pulsing
>> activity but you can get away with less frames as you don't need to
>>   >  redraw anything during the hold.
>>
>>   >  This doesn't effect the initial start-up however, which I
>> believe was the original ticket... FWIW I think F14 builds have
>>   >  regressed for some other reason here, perhaps it's the window
>> manager being laggy (I occasionally see window bezels before
>>   >  things go full screen).
>>
>>   >  Regards,
>>   >  --Gary"
>>
>> --But we realised that any changes made to improve the animation were
>> not affecting the initial starting time , which is our main
>>  point of concern.
>
> This is not a correct summary of what Gary said - he didn't say "any
> changes...".  I think you menat something else, because it also
> doesn't agree with how you started point 3#.
I started this note following the format :

s.no  |            <HEADING(Hint of  the comment given below)>
        |            <PREVIOUS COMMENTS OF A DEVELOPER>
        |            <END RESULTS DEDUCED BY ME(not  a summary)>
so the end results are not always meant to support the heading.



>> 4) Prerendering the pulsing icon and maintaining a cache (As said
>> earlier in point #1 also) type scenario for the different frames of
>>    the animation also looks like a plausible option:
> [...]
>> --We can certainly try this option as it can possibly solve the
>> problem to a large extent and improve the system performance.
>
> Again, this is already done.
>
>> Regards,
>>
>> --Anurag
>
> Martin
>