<html><body bgcolor="#FFFFFF"><div>Hi Gonzalo,<br><br>On 12 Aug 2011, at 16:25, Gonzalo Odiard <<a href="mailto:gonzalo@laptop.org">gonzalo@laptop.org</a>> wrote:<br><br></div><div></div><blockquote type="cite"><div>Gary,<br>We are looking if this can land one of this in 0.94<br>I am proposing land the first version, already reviewed by Simon and Sasha,<br>and improve it later with your suggestion.<br>In fact, I think what you are proposing is simpler than what I did.<br>
Is this a good alternative for you?<br></div></blockquote><div><br></div><div>+1, with the understanding that we do try to land the improved version at some point and not let it fall off our radar :)</div><div><br></div><div>--Gary</div><br><blockquote type="cite"><div>Gonzalo<br><br><div class="gmail_quote">On Thu, Aug 11, 2011 at 2:24 PM, Gary Martin <span dir="ltr"><<a href="mailto:garycmartin@googlemail.com"><a href="mailto:garycmartin@googlemail.com">garycmartin@googlemail.com</a></a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hi Gonzalo,<br>
<div class="im"><br>
On 5 Aug 2011, at 20:47, Gonzalo Odiard wrote:<br>
<br>
> I have prepared a few videos to show the change in the startup animation I am proposing<br>
> In all the videos, at the right is one XO with the patch and the left is another without the patch,<br>
> the method to compare the times is not very scientific, but I have tried to start the activities at the same time :)<br>
><br>
> In the first test, I used alpha and scale cairo operations to modify to render only one time the icon<br>
> you can see the zoom effect at the start and a little improvement in startup time:<br>
><br>
> <a href="http://www.youtube.com/watch?v=dE7eXjsQm_g" target="_blank"><a href="http://www.youtube.com/watch?v=dE7eXjsQm_g">http://www.youtube.com/watch?v=dE7eXjsQm_g</a></a><br>
><br>
> <a href="http://www.youtube.com/watch?v=nHqcdsAFvgE" target="_blank"><a href="http://www.youtube.com/watch?v=nHqcdsAFvgE">http://www.youtube.com/watch?v=nHqcdsAFvgE</a></a><br>
><br>
> <a href="http://www.youtube.com/watch?v=g4qMB4eayU4" target="_blank"><a href="http://www.youtube.com/watch?v=g4qMB4eayU4">http://www.youtube.com/watch?v=g4qMB4eayU4</a></a><br>
<br>
</div>Fab! That's a good 2 or 3 seconds shaved off activity start time, a very important metric, even given we loose the original pulse design (i.e. pulsing between grey+white & user colours provides better contrast; the monochrome to colour also suggests an activity bundle starting up and becoming a users own activity).<br>
<br>
Are these two test machines XO-1, or XO-1.5s?<br>
<br>
Regarding the zoom effect, it's great it is finally showing up ;-) but I wonder if the timing is different, it feels very slow/smooth/linear, is it using the same zoom maths as on previous Sugars? I'll need to have a tinker to see/check how it used to be.<br>
<div class="im"><br>
> If you want test in your system, you can download <a href="http://dev.laptop.org/%7Egonzalo/alpha-pulsingicon/files_animation.zip" target="_blank"><a href="http://dev.laptop.org/~gonzalo/alpha-pulsingicon/files_animation.zip">http://dev.laptop.org/~gonzalo/alpha-pulsingicon/files_animation.zip</a></a><br>
> and copy the files in the directory pulse in the following directories:<br>
> icon.py -> /usr/lib/python/python2.7/site-packages/sugar/graphics/<br>
> launcher.py and pulsingicon.py -> /usr/lib/python/python2.7/site-packages/jarabe/view/<br>
><br>
> The patches are:<br>
> <a href="http://dev.laptop.org/%7Egonzalo/alpha-pulsingicon/Add-alpha-and-scale-functionality-to-the-icon-can-be-used-in-the-launcher-to-do-the-animation---v2.patch" target="_blank"><a href="http://dev.laptop.org/~gonzalo/alpha-pulsingicon/Add-alpha-and-scale-functionality-to-the-icon-can-be-used-in-the-launcher-to-do-the-animation---v2.patch">http://dev.laptop.org/~gonzalo/alpha-pulsingicon/Add-alpha-and-scale-functionality-to-the-icon-can-be-used-in-the-launcher-to-do-the-animation---v2.patch</a></a><br>
> <a href="http://dev.laptop.org/%7Egonzalo/alpha-pulsingicon/0001-Do-startup-animation-of-the-activity-icon-using-scal.patch" target="_blank"><a href="http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Do-startup-animation-of-the-activity-icon-using-scal.patch">http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Do-startup-animation-of-the-activity-icon-using-scal.patch</a></a><br>
><br>
> You can see the effect is different, and may be is not so good like the original.<br>
><br>
> I did another try using two buffers, with the different colors, and changing the alpha<br>
> to obtain a similar effect. The code is a lot more complex, and is a little slower obviously.<br>
><br>
> <a href="http://www.youtube.com/watch?v=9rLaE0TKZXw" target="_blank"><a href="http://www.youtube.com/watch?v=9rLaE0TKZXw">http://www.youtube.com/watch?v=9rLaE0TKZXw</a></a><br>
><br>
> <a href="http://www.youtube.com/watch?v=-Q2RliMTLRM" target="_blank"><a href="http://www.youtube.com/watch?v=-Q2RliMTLRM">http://www.youtube.com/watch?v=-Q2RliMTLRM</a></a><br>
><br>
> <a href="http://www.youtube.com/watch?v=SHBCSnrGsjA" target="_blank"><a href="http://www.youtube.com/watch?v=SHBCSnrGsjA">http://www.youtube.com/watch?v=SHBCSnrGsjA</a></a><br>
<br>
</div>I do find these more visible as the grey/white stage of the pulse is guaranteed to provide some contrast even if the user has chosen a fill/stroke combination with low contrast. However, the cost seems rather high with only about 1sec launch time improvement and twice as much code to maintain.<br>
<br>
Quick question about your second version (I couldn't quite grok my way through the patch). Are you 1) fading both the grey and the colour versions of the image so that as one becomes opaque, the other becomes transparent; or 2) drawing one in the background always as an opaque fill, and then just pulsing the other over the top from opaque to transparent? In my code (non Sugar) I find the latter choice keeps code to a minimum, is more efficient, and visually looks cleaner (avoids the white background leaking through during part of the cycle).<br>
<br>
Here's a quick video if I didn't describe the above effect well enough (middle is alpha fading both images, right is the simple case of a single fade in/out for comparison, and left is using the monochrome image as an opaque background fill while drawing the colour image over the top with an alpha puse):<br>
<br>
<a href="http://youtu.be/LLsqjGWBMfQ" target="_blank"><a href="http://youtu.be/LLsqjGWBMfQ">http://youtu.be/LLsqjGWBMfQ</a></a><br>
<br>
Regards,<br>
<font color="#888888">--Gary<br>
</font><div class="im"><br>
> The patchs are:<br>
><br>
> <a href="http://dev.laptop.org/%7Egonzalo/alpha-pulsingicon/0001-Add-alpha-to-icon-and-secondary-buffer-to-implement-.patch" target="_blank"><a href="http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Add-alpha-to-icon-and-secondary-buffer-to-implement-.patch">http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Add-alpha-to-icon-and-secondary-buffer-to-implement-.patch</a></a><br>
><br>
> <a href="http://dev.laptop.org/%7Egonzalo/alpha-pulsingicon/0001-Do-startup-animation-using-alpha-scale-and-a-seconda.patch" target="_blank"><a href="http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Do-startup-animation-using-alpha-scale-and-a-seconda.patch">http://dev.laptop.org/~gonzalo/alpha-pulsingicon/0001-Do-startup-animation-using-alpha-scale-and-a-seconda.patch</a></a><br>
><br>
> If you want test in one XO can use the files in the directory pulse-doublebuf in the files_animation.zip<br>
><br>
> I want include this in 0.94, thoughts?<br>
><br>
> Gonzalo<br>
><br>
</div><div><div></div><div class="h5">> _______________________________________________<br>
> Sugar-devel mailing list<br>
> <a href="mailto:Sugar-devel@lists.sugarlabs.org"><a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a></a><br>
> <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank"><a href="http://lists.sugarlabs.org/listinfo/sugar-devel">http://lists.sugarlabs.org/listinfo/sugar-devel</a></a><br>
<br>
</div></div></blockquote></div><br>
</div></blockquote></body></html>