Ticket #7356 (closed defect: fixed)

Opened 5 months ago

Last modified 2 months ago

regression in activity view performance.

Reported by: cscott Owned by: tomeu
Priority: normal Milestone: 8.2.0 (was Update.2)
Component: sugar Version: Update.1
Keywords: 8.2.0:? relnote r+ Cc: benzea, seberg
Action Needed: no action Verified: no
Deployments affected: Blocked By:
Blocking:

Description

The activity view takes a *long* time to un-overlap its icons in joyride-2073.

Attachments

Change History

Changed 5 months ago by marco

  • owner changed from marco to tomeu

Changed 5 months ago by tomeu

  • keywords 8.2.0:? added
  • next_action set to communicate

Does this happened on the first boot after olpc-updating? If so, there may have been the jffs2 thread taking up all cpu.

Changed 4 months ago by tomeu

  • keywords relnote added
  • status changed from new to closed
  • resolution set to invalid

We should add to the release notes that the first one or two minutes after an update, the system will be slower than normal.

Changed 4 months ago by mstone

  • status changed from closed to reopened
  • next_action changed from communicate to finalize
  • resolution deleted

Please indicate the desire for release-notes by supplying some sample text and by placing the ticket in the 'finalize' state. (Use of the relnote keyword is a fine hint.) The release team will close the ticket when it is properly noted.

Changed 3 months ago by seberg

  • cc benzea added

As /view/home/grid.py uses numpy in any case, the nested for loops to calculate weight (or update the weight array) that are computed quite often can be spead up by using numpy slicing + numpy summing. I will attach a patch. The patch also currently changes _MAX_COLLISIONS_PER_REFRESH, I am not sure if this is good or not.

Changed 3 months ago by seberg

  • keywords r? added

Changed 3 months ago by seberg

Changed 3 months ago by seberg

  • cc seberg added

Changed 3 months ago by tomeu

  • keywords r+ added; r? removed
  • next_action changed from finalize to review

Changed 3 months ago by benzea

  • next_action changed from review to package

OK, I just pushed a cleaned up version to both master and sucrose-0.82 for my brother.

Changed 3 months ago by marco

  • next_action changed from package to test in build

Changed 3 months ago by seberg

|TestCase|

Try dragging two icons on top of each other (or switching between freeform and ring layout when there are overlaps created through drag and dropping, etc.) and it should react faster with the changes.

Changed 3 months ago by marco

  • next_action changed from test in build to add to build

Changed 3 months ago by marco

  • next_action changed from add to build to approve for release

Changed 3 months ago by marco

  • next_action changed from approve for release to add to build

Changed 3 months ago by cscott

  • next_action changed from add to build to add to release

Exact package name & version, please.

Changed 3 months ago by marco

sugar-artwork-0.82.1-1.fc9

Changed 3 months ago by marco

Sorry, but copy and paste, it's really:

sugar-0.82.2-1.fc9

Changed 3 months ago by cscott

  • next_action changed from add to release to test in release

Committed to stable repo: http://mock.laptop.org/gitweb/gitweb.cgi?p=repos;a=commitdiff;h=e0489cd6dec4b910445f489632adb44ba136c1c1

Should be in build 758 and following; please test.

Changed 2 months ago by erikos

  • status changed from reopened to closed
  • next_action changed from test in release to no action
  • resolution set to fixed

looks good to me

Changed 2 months ago by frances

looks good in both 8.2-759 and 760

Note: See TracTickets for help on using tickets.