Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
I again tried the patch on my XO-1.5 build-0.88.1 but with no improvement. I made and saved the changed as written in your patch in the pulsingicon.py file and restarted the sugar session to let the changes take place but it didnt offer any changes for the good. I hope we can test your patch on more XO devices to verify if its a device specific problem. On Fri, Nov 5, 2010 at 11:42 PM, Martin Dengler mar...@martindengler.com wrote: On Fri, Nov 05, 2010 at 10:59:05PM +0530, Anurag Chowdhury wrote: Then I may have misinterpreted Gary's comment.I apologize in that case. But when I tested Martin's patch on my XO-1.5 , 0.88.1 build , upon starting an activity the pulsing icon animation was replaced by a static grayscale svg icon of the activity ,whose screenshots I have placed at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds. To reach to the above results I applied the changes suggested at http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch Please let me know if I needed to add any more changes to get the patch working because the sole application of this patch doesn't produce the said changes at my end. I just tried it on 0.88-1 in the emulator and it worked fine. At least one other person tried it on HEAD (as I did before) and it worked fine. Can you try it again please, or let me know how you're applying the patch? Martin ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 5, 2010 at 12:46 PM, Gonzalo Odiard gonz...@laptop.org wrote: [snip] Is really needed the zoom? If we can't do it in a efficient way, may be we can use another metaphor. What about moving the icon from the position in the home view to the screen center? Or perhaps a simple orbit. I think we must do all we can to start quickly the activities. +1 Gonzalo ___ Dextrose mailing list dextr...@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/dextrose -walter -- Walter Bender Sugar Labs http://www.sugarlabs.org ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, 2010-11-05 at 13:46 -0300, Gonzalo Odiard wrote: Is really needed the zoom? If we can't do it in a efficient way, may be we can use another metaphor. What about moving the icon from the position in the home view to the screen center? I think we must do all we can to start quickly the activities. +1. We should *also* file bugs for the activities with icons that are very slow to render. This affects negatively also startup of Sugar itself, when such activities are displayed in the home view. -- // Bernie Innocenti - http://codewiz.org/ \X/ Sugar Labs - http://sugarlabs.org/ ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 05, 2010 at 01:46:27PM -0300, Gonzalo Odiard wrote: Hmmm, no, with Martin's below linked patch pulse colour animation is good. The only extra thing I noted in my email was that the 1sec zoom effect is still not showing for anything by very simple icons (Log icon is simple enough, Distance is not), however the zoom effect is not showing with your 'skip the first update' approach either. Fixing zoom efficiency should be looked at as a separate issue. Is really needed the zoom? If we can't do it in a efficient way, may be we can use another metaphor. What about moving the icon from the position in the home view to the screen center? I think we must do all we can to start quickly the activities. I agree - I don't see any point in a zoom that's not shown. It seems like we might be able to get it quick enough before the next freeze, though. If not I wouldn't object to removing it. Gonzalo Martin pgpej9n3Jr4UN.pgp Description: PGP signature ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
Then I may have misinterpreted Gary's comment.I apologize in that case. But when I tested Martin's patch on my XO-1.5 , 0.88.1 build , upon starting an activity the pulsing icon animation was replaced by a static grayscale svg icon of the activity ,whose screenshots I have placed at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds. To reach to the above results I applied the changes suggested at http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch Please let me know if I needed to add any more changes to get the patch working because the sole application of this patch doesn't produce the said changes at my end. Also I would wish to ask for Gary's experience after he applied my patch which I think he once referred to in the previous issue thread. I worked on the pointers provided there by Gary and would love further suggestions. On Fri, Nov 5, 2010 at 10:09 PM, Gary Martin garycmar...@googlemail.com wrote: Hi Anurag, On 5 Nov 2010, at 16:19, Anurag Chowdhury anu...@seeta.in wrote: On Fri, Nov 5, 2010 at 6:53 PM, Martin Dengler mar...@martindengler.com wrote: On Fri, Nov 05, 2010 at 06:05:16PM +0530, Anurag Chowdhury wrote: Sure , we can work to get a combined patch but your patch still disables the animation in most of the icons I think you're talking about something else than my patch. No animation is disabled. Not sure what's wrong but the patch you linked to[1] in your wiki page clearly does nothing like disabling any animation. Well, I actually didn't meant that. I was talking about the change noticed by Gary Hmmm, no, with Martin's below linked patch pulse colour animation is good. The only extra thing I noted in my email was that the 1sec zoom effect is still not showing for anything by very simple icons (Log icon is simple enough, Distance is not), however the zoom effect is not showing with your 'skip the first update' approach either. Fixing zoom efficiency should be looked at as a separate issue. --Gary (and me too) i.e. after the application of the patch we don't get to see the pulsing icon animation in most of the icons , all we see is only the grayscale svg icon , this anomaly is also linked with the complexity of the icons i.e. more complex icons don't show up the animation while the simpler ones do. Regards Anurag Martin 1. http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch ___ Dextrose mailing list dextr...@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/dextrose ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 05, 2010 at 10:59:05PM +0530, Anurag Chowdhury wrote: Then I may have misinterpreted Gary's comment.I apologize in that case. But when I tested Martin's patch on my XO-1.5 , 0.88.1 build , upon starting an activity the pulsing icon animation was replaced by a static grayscale svg icon of the activity ,whose screenshots I have placed at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds. To reach to the above results I applied the changes suggested at http://www.martindengler.com/tmp/sl.o-2080/pulsingicon.py-set-new-colors-in-one-go-to-avoid-multiple-calls-to-SVG-rendering.patch Please let me know if I needed to add any more changes to get the patch working because the sole application of this patch doesn't produce the said changes at my end. I just tried it on 0.88-1 in the emulator and it worked fine. At least one other person tried it on HEAD (as I did before) and it worked fine. Can you try it again please, or let me know how you're applying the patch? Martin pgpXvkNBrU5Zi.pgp Description: PGP signature ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [Dextrose] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, 2010-11-05 at 17:53 +, Gary Martin wrote: The loss of the zoom animation for non-trivial icons is a regression, it was fine 2 years back with the OLPC build 767, sugar 0.82.1 and Fedora 9 (just retested here on an XO-1). We upgraded cairo and pixman in Dextrose to the versions shipped with Fedora 14. There was no time to run a benchmark, but it seems a lot faster now. Still not fast enough to see the zoom animation, though. Dragging Marco in: do you know if something was changed in Sugar or librsvg that could have made impacted performance? -- // Bernie Innocenti - http://codewiz.org/ \X/ Sugar Labs - http://sugarlabs.org/ ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
