[Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
Hello I have prepared a detailed report on the issue SL # 2080, Pulsing icon delayed by 5 seconds or so , and have made it online at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds . The report includes detailed analysis and developments on the issue till date and various proposed fixes and their end results. I hope the report will help us better to monitor the issue. Any feedback is appreciated. Regards, --Anurag Research Engineer SEETA ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
[Sugar-devel] [RELEASE] Pippy-38
== Source == http://download.sugarlabs.org/sources/sucrose/fructose/Pippy/Pippy-38.tar.bz2 == News == * Fix svg icons for Pippy * Add EditToolbar modify toolbars for =0.84 compatibility (Gonzalo and Anish) * Remove the 'dobject'/groupthink submodule * Switch activity.info from class to exec NOTE: When the activity is shared, the 'undo' and 'redo' buttons are made insensitive. signature.asc Description: This is a digitally signed message part ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
Sure , we can work to get a combined patch but your patch still disables the animation in most of the icons and we can check the real deal only after we can see the animation because thats the real part where all the system resources go and lead to that delay. We should first try to get that part working and then run fresh tests to check the improvements. On the other hand my patch my patch also gives a near 50% reduction in delay with the animation playing but the community still hasn't reached on a consensus regarding the issue so I guess we need to reach on a common point of approval , then only we can get any working patch accepted and move on to other issues. Regards Anurag On Fri, Nov 5, 2010 at 4:05 PM, Martin Dengler mar...@martindengler.com wrote: On Fri, Nov 05, 2010 at 02:24:58PM +0530, Anurag Chowdhury wrote: Hello I have prepared a detailed report on the issue SL # 2080, Pulsing icon delayed by 5 seconds or so , and have made it online at http://wiki.sugarlabs.org/go/Pulsing_icon_delayed_by_5_seconds . Thanks for that. Next time, could you link to the SL bug in the wiki page and the wiki page in the SL bug? I have done this for you. Any feedback is appreciated. Again, thanks for this summary. I think the thing to do is merge my patch for #2080 and then address the other issues. I'll do that soon. Regards, --Anurag Martin ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
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. 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 pgpTNJcvBAYTE.pgp Description: PGP signature ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 5, 2010 at 6:35 AM, Martin Dengler mar...@martindengler.com wrote: Again, thanks for this summary. I think the thing to do is merge my patch for #2080 and then address the other issues. I'll do that soon. Hi Martin. would it make sense to change (in toolkit) jarabe/view/icon.py to optionally return the cairo object, instead of paint()ing it? Then you can render the SVG once or twice (for zoom effect?), keep those raster images cached in the pulsingicon object, and use Cairo's paint_with_alpha() instead of straight paint(). Which will be fast. (Load/render your sprites only once, get creative in how you blit them... ) The pulsing color combinations change (get simpler) with this. To get back to what we have now (or an approximation) you could render to blackwhite (and a reverse) and use them as masks, painting with masked full color fills. A bit more work. I'm following this tutorial -- mask() and paint_with_alpha() should be enough. I am disappointed that I cannot see more advanced blitting opts. http://www.tortall.net/mu/wiki/CairoTutorial m -- martin.langh...@gmail.com mar...@laptop.org -- School Server Architect - ask interesting questions - don't get distracted with shiny stuff - working code first - http://wiki.laptop.org/go/User:Martinlanghoff ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 05, 2010 at 09:28:23AM -0400, Martin Langhoff wrote: On Fri, Nov 5, 2010 at 6:35 AM, Martin Dengler mar...@martindengler.com wrote: Again, thanks for this summary. I think the thing to do is merge my patch for #2080 and then address the other issues. I'll do that soon. If I call this change #1... would it make sense to change (in toolkit) jarabe/view/icon.py to optionally return the cairo object, instead of paint()ing it? Then you can render the SVG once or twice (for zoom effect?), keep those raster images cached in the pulsingicon object, and use Cairo's paint_with_alpha() instead of straight paint(). Which will be fast. (Load/render your sprites only once, get creative in how you blit them... ) The pulsing color combinations change (get simpler) with this. ...and this change #2... To get back to what we have now (or an approximation) you could render to blackwhite (and a reverse) and use them as masks, painting with masked full color fills. A bit more work. ...and this change #3, I'll have a go at implementing #2 and #3 soon. Since #1 is separate and an improvement from what we have now, it can be merged earlier9 Unless there are issues, which I haven't seen yet). I'm following this tutorial -- mask() and paint_with_alpha() should be enough. I am disappointed that I cannot see more advanced blitting opts. http://www.tortall.net/mu/wiki/CairoTutorial Thanks for the link - I've seen it before but it's always good to come back to absorb a bit more. m Martin pgpTOVmf91qPQ.pgp Description: PGP signature ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
[Sugar-devel] [ASLO] Release Pippy-38
Activity Homepage: http://activities.sugarlabs.org/addon/4041 Sugar Platform: 0.82 - 0.90 Download Now: http://activities.sugarlabs.org/downloads/file/27097/pippy-38.xo Release notes: Changelog * Fix svg icons for Pippy * Add EditToolbar modify toolbars for Sugar Labs Activities http://activities.sugarlabs.org ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
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 (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 ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] Report on Sl# 2080 , Pulsing icon delayed by 5 seconds or so.
On Fri, Nov 05, 2010 at 09:49:01PM +0530, Anurag Chowdhury 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 (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. Are you talking about the pulsing from greyscale to color, or the zooming? Gary was talking about the zooming[1] IIUC, and that is obviously not changed by my patch. I understood his comment to mean that there was already a zooming issue in the build he was referring to, and that my patch did not change that. Can you please help me understand a bit more why you think my patch causes the pulsing icon to not show in color? I don't see that in my sugar-jhbuild before or after my patch. Martin 1. http://lists.sugarlabs.org/archive/sugar-devel/2010-October/028444.html pgptiHCsoKp3T.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, 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
[Sugar-devel] Help testing Write
If you can, would be great have feedback about the last changes in Write. I have uploaded a activity ONLY FOR TEST to http://dev.laptop.org/~gonzalo/Write-73.xo This version is for Sugar 0.84, if want to help testing with Sugar 0.84, the next week we will have versions for 0.84 Features to test: 1) Write: Change the default method of insert images By default Write inserted the images floating, I have changed it to insert the image in line. Now you can insert a image in a table and in my tests, the images are easier to use. If you want insert a image floating, there are a new palette, in the insert image button with a checkbox to enable it. May be is not the best interface, suggestions welcomed. 2) Cleanup sharing code Updated and applied patch from tickets SL #1962, OLPC #5062, OLPC #8680. There are not effects to the end user. If there are any regression would be informed. 3) Change the logic for preserve the mime type at reading and saving files Previously, Write did wrong things when opened text files, or when you open files from a usb drive and copy again the files in the usb. There are tickets filled: SL #2127, OLPC #8972, OLPC #5291 and OLPC #1925 Now you can open and save from every supported file types (**) without problems. If you save a .doc file, will change the extension to rtf, because Abiword does not save .doc files If you copy a .abw file to the usb, will be saved like .abw.CRASHED but it is related to anther problem (in the Journal code) anyway you can open the file created. 4) Fix paste images in write - olpc #2507 and olpc #7186 Now you can copy a image from Paint, Browse or Write and paste in Write You need the last versions of Browse and Paint. Thanks! Gonzalo ** supported mimetypes: text/rtf;text/plain;application/x-abiword;text/x-xml-abiword;application/msword;application/rtf;application/xhtml+xml;text/html;application/vnd.oasis.opendocument.text ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
[Sugar-devel] [DESIGN] Messages notification
Hello amigos, Recently I encountered many situations where a system-wide notification-messages system is required as a basis for many bug fixes or enhancements. I am not talking about the messages that are related to an specific activity or its context, in many of those cases the current alert widgets are effective and proper methods. Currently, Sugar does not have a mechanism to communicate to users different kind of information about sugar itself. For example: 1. Important system events (automatic backup running, new update available, journal being almost full,etc) 2. Connectivity status changes or errors (connecting to a wifi network or 3G) 3. Critical sugar errors, right now most of those errors are invisible to users (this make testing process very difficult) 4. I am sure others can imagine more use cases. So I would like to list two extensions to the current notification system. Basically what we need is: 1. Simple text messages display. 2. Messages manager, this would help the user to see what messages have been delivered in the current session, so they could react to those events (for example, reporting bugs). I believe this feature could be a huge plus to dextrose and and sugar mainstream, therefore I invite everyone to share their ideas on how this feature could work and look. Saludos Martin (tch) Abente ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
Re: [Sugar-devel] [DESIGN] Messages notification
On Fri, Nov 05, 2010 at 06:02:28PM -0300, Martin Abente wrote: Hello amigos, Recently I encountered many situations where a system-wide notification-messages system is required as a basis for many bug fixes or enhancements. [...] Currently, Sugar does not have a mechanism to communicate to users different kind of information about sugar itself. What about the notification system? The Log activity? The Journal Full warning? The Frame? There are a lot of communication mechanisms, but they dn't seem to do what you wnat: 1. Simple text messages display. Where do you think this would best be shown? Text is limited in the UI to avoid literacy and translation burdens. 2. Messages manager, this would help the user to see what messages have been delivered in the current session, so they could react to those events (for example, reporting bugs). Peraps Log could be changed to show this? Or the Frame (doesn't it already show the last few notifications?)? Saludos Martin (tch) Abente Martin pgpLccdLePGH3.pgp Description: PGP signature ___ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel