[Digikam-devel] [digikam] [Bug 318516] Improve digiKam loading time [patch]

[Yann LE HIR]

https://bugs.kde.org/show_bug.cgi?id=318516

Yann LE HIR <yann.lehir at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #78993|0                           |1
        is obsolete|                            |

--- Comment #7 from Yann LE HIR <yann.lehir at gmail.com> ---
Created attachment 79013
  --> https://bugs.kde.org/attachment.cgi?id=79013&action=edit
Cleaner patch  to change the loading sequence and keep the session restore.

I joined a cleaner patch for the loading sequence.

On a clean branch, with digikam compiled in debug mode.
I get an average loading time of 33.36s (with 10 launches)
With the patches I get an average loading time of 2.85s
Almost 12x faster :)
It's near 10x faster in release mode.

I'm pretty sure it was the main issue. The fact that you try to reload 16k
times all the filter models is quite a performance killer. Not to mention the
AbstractAlbumModel that also took something like 32k events (AboutToBeAdded +
Added)...
Without the patches, profiling on the loading of digikam can show up to 103
million calls to Album::next() and a fair amount of Album::title()...

-- 
You are receiving this mail because:
You are the assignee for the bug.