vauvenal5 / yaga

Nextcloud Yaga - A Nextcloud first gallery app for Android.
https://vauvenal5.github.io/yaga-docs/
GNU General Public License v3.0
136 stars 9 forks source link

Performance optimization #2

Open vauvenal5 opened 3 years ago

vauvenal5 commented 3 years ago

Loading big lists of images needs further optimization. Two obvious issues are that we are sorting the image list in the main thread and that we are merging lists in the main thread.

jakobroehrl commented 3 years ago

With 500 pics the reload time is about 3 secs, this is ok for me. But If I show all of my pics (10000) I can't wait till a realod is done.

vauvenal5 commented 3 years ago

This is actually already considered fixed. Since the two operations mentioned here are already in the background thread.

As far as I know there is currently no pagination implemented in Nextcloud, so I do not have any choice then loading the full list of images. Not sure that I can do anything more here.

What would be the expected behavior from your point of view?

vauvenal5 commented 3 years ago

See also this https://github.com/vauvenal5/yaga/issues/80

vauvenal5 commented 3 years ago

And this: https://github.com/nextcloud/ios/issues/845