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

[Martin Dengler]

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#.

> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20101031/0c32845f/attachment.pgp>