Ubuntu
unity8 package

Slow loading/showing of images in the dash

Bug #1338430 reported by Thomas Strehl
82
This bug affects 10 people
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Fix Released
Critical
Albert Astals Cid

Bug Description

Installed app icons (when expanding the category) show very slowly after reboot. It actually makes the impression that those are being downloaded.

Related branches

lp:~unity-team/unity8/croppedImageMinimumSourceSize
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Mirco Müller (community): Approve
Albert Astals Cid (community): Abstain (is my code)
Lars Karlitski: Pending requested
Andrea Cimitan: Pending requested
Diff: 239 lines (+55/-28)
9 files modified
plugins/Dash/CardCreator.js (+4/-7)
plugins/Dash/CroppedImageMinimumSourceSize.qml (+38/-0)
plugins/Dash/qmldir (+2/-1)
tests/plugins/Dash/cardcreator/1.res (+2/-2)
tests/plugins/Dash/cardcreator/2.res (+2/-5)
tests/plugins/Dash/cardcreator/3.res (+2/-2)
tests/plugins/Dash/cardcreator/4.res (+1/-4)
tests/plugins/Dash/cardcreator/5.res (+2/-2)
tests/plugins/Dash/cardcreator/7.res (+2/-5)
lp:~unity-team/unity8/scopes_more_things_on_memory
Andrea Cimitan (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Albert Astals Cid (community): Abstain (has code of mine)
Diff: 1223 lines (+458/-95)
42 files modified
build.sh (+1/-0)
cmake/modules/QmlTest.cmake (+2/-1)
debian/control (+1/-0)
plugins/Dash/CardCreator.js (+5/-2)
plugins/Dash/listviewwithpageheader.cpp (+18/-7)
plugins/Dash/listviewwithpageheader.h (+6/-0)
qml/Components/ResponsiveGridView.qml (+5/-1)
qml/Dash/CardGrid.qml (+1/-0)
qml/Dash/CardTool.qml (+8/-5)
qml/Dash/CardVerticalJournal.qml (+1/-0)
qml/Dash/DashContent.qml (+3/-0)
qml/Dash/DashNavigation.qml (+1/-0)
qml/Dash/DashRenderer.qml (+4/-0)
qml/Dash/GenericScopeView.qml (+37/-14)
qml/Dash/ScopeListView.qml (+1/-0)
qml/Dash/ScopesOverviewAll.qml (+2/-0)
tests/autopilot/unity8/shell/emulators/dash.py (+2/-2)
tests/mocks/Ubuntu/CMakeLists.txt (+1/-0)
tests/mocks/Ubuntu/Connectivity/CMakeLists.txt (+10/-0)
tests/mocks/Ubuntu/Connectivity/networking-status.cpp (+64/-0)
tests/mocks/Ubuntu/Connectivity/networking-status.h (+73/-0)
tests/mocks/Ubuntu/Connectivity/plugin.cpp (+46/-0)
tests/mocks/Ubuntu/Connectivity/plugin.h (+35/-0)
tests/mocks/Ubuntu/Connectivity/qmldir (+2/-0)
tests/plugins/Dash/CMakeLists.txt (+3/-3)
tests/plugins/Dash/cardcreator/1.res (+2/-1)
tests/plugins/Dash/cardcreator/2.res (+2/-1)
tests/plugins/Dash/cardcreator/3.res (+3/-2)
tests/plugins/Dash/cardcreator/4.res (+3/-2)
tests/plugins/Dash/cardcreator/5.res (+3/-2)
tests/plugins/Dash/cardcreator/7.res (+3/-2)
tests/plugins/Dash/listviewwithpageheadertest.cpp (+39/-0)
tests/plugins/Dash/listviewwithpageheadertest.qml (+1/-0)
tests/plugins/Dash/listviewwithpageheadertestsection.qml (+1/-0)
tests/plugins/Dash/listviewwithpageheadertestsectionexternalmodel.qml (+1/-0)
tests/plugins/Ubuntu/Gestures/CMakeLists.txt (+1/-1)
tests/qmltests/Components/CMakeLists.txt (+1/-1)
tests/qmltests/Dash/tst_Card.qml (+26/-0)
tests/qmltests/Dash/tst_Dash.qml (+16/-39)
tests/qmltests/Dash/tst_DashContent.qml (+14/-1)
tests/qmltests/Dash/tst_GenericScopeView.qml (+7/-6)
tests/utils/modules/Unity/Test/UnityTestCase.qml (+3/-2)
lp:ubuntu/utopic-proposed/unity8
Revision history for this message
Michał Sawicz (saviq) wrote :

Hmm, they can't be downloaded since they're all local paths, either files directly or theme-based.

We want to disable delegate management on the apps page so that they're always in memory, investigation needs to happen as to why they would load slowly in any case.

Revision history for this message
Michał Sawicz (saviq) wrote :

Ah btw, they load faster afterwards because they're in memory already.

Revision history for this message
Thomas Strehl (strehl-t) wrote :

Yes, kind of. Just doing a quick full swipe down and up on installed apps scope will also trigger reloading of icons which is also very noticeable. Should we cache more aggressively?

Thomas Strehl (strehl-t)
tags: added: rtm14
Florian Boucault (fboucault)
Changed in unity8:
status: New → Confirmed
Revision history for this message
Michał Sawicz (saviq) wrote :

There's a few things we need to find out:

a) why the UI thread is blocked when loading images in dash
b) requests to image providers are not parallel (Qt limitation), are we going through any?
c) we should just create all the delegates as you switch to a scope, only defer loading the images

Michał Sawicz (saviq)
summary: - Slow loading/showing of installed app icons
+ Slow loading/showing of images in the dash
Michał Sawicz (saviq)
Changed in unity8:
status: Confirmed → Triaged
importance: Undecided → Critical
Michał Sawicz (saviq)
Changed in unity8:
assignee: nobody → Albert Astals Cid (aacid)
status: Triaged → In Progress
kevin gunn (kgunn72)
tags: added: touch-2014-09-25
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → Critical
status: New → In Progress
Michał Sawicz (saviq)
no longer affects: unity8
Revision history for this message
Rick Spencer (rick-rickspencer3) wrote :

This does not happen to me only after reboot. When I scroll up and down in the apps scope there is a delay in loading icons.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Confirmed, the icons are very slow to appear all the time on N4. Not just after reboot.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

About 3 seconds is the time it takes to show all the icons after I scroll down and then back up to the top again in the apps scope.

It doesn't look like a blocked UI thread at all. Rather we're unloading/reloading some images during scrolling. Surely if you want smooth scrolling you need to keep everything in memory?..

Stephen M. Webb (bregma)
tags: added: touch-2014-10-09
removed: touch-2014-09-25
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.00+14.10.20141006-0ubuntu1

---------------
unity8 (8.00+14.10.20141006-0ubuntu1) utopic; urgency=low

  [ Andrea Cimitan ]
  * Tweak card header to match the spec
  * Add preview image slideshow (LP: #1351537)

  [ Michał Sawicz ]
  * Cache more things in memory, so flicking scopes should be faster
    (LP: #1336724)
  * Tweak card header to match the spec
  * Save texture memory by limiting sourceSize (LP: #1338430)

  [ Ying-Chun Liu ]
  * Add attributes to Preview. (LP: #1282460)

  [ Albert Astals ]
  * Update pot
  * Cache more things in memory, so flicking scopes should be faster
    (LP: #1336724)
  * Save texture memory by limiting sourceSize (LP: #1338430)
  * Clip the settings list
  * Fix unlocking from the left again
  * Add wait_ makes tests more reliable

  [ Michael Zanetti ]
  * fix fading out the launcher instead of sliding it out on left-edge
    minimizing an app.
  * Make the DashCommunicator async and more flexible to handle a
    lifecycle-suspended dash (LP: #1339883)

  [ Michael Terry ]
  * Retry unlock-device script if it fails, as there is always a risk of
    a small race with boot-up. (LP: #1370644)
  * Add pull-to-refresh functionality to scopes. (LP: #1368336)

  [ CI bot ]
  * Resync trunk
 -- Ubuntu daily release <email address hidden> Mon, 06 Oct 2014 08:03:23 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.