Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.0RC1: Favorites and Photos loading for ever #1972

Closed
steve-s-71 opened this issue Jan 16, 2018 · 14 comments · Fixed by #2250
Closed

3.0RC1: Favorites and Photos loading for ever #1972

steve-s-71 opened this issue Jan 16, 2018 · 14 comments · Fixed by #2250

Comments

@steve-s-71
Copy link

When selecting menu point "Favorites" or "Photos", the app shows "loading" and the "sandbox" circle does not stop spinning. No content is shown.

@xXSTrikeXx
Copy link
Contributor

xXSTrikeXx commented Jan 18, 2018

It´s the same when clicking in the menu on photos.

@tobiasKaminsky
Copy link
Member

This is a known problem, if there are too many items.
For 5k images:

  • fetch from server ~30s (as the server has to gather all information on each image/file)
  • displaying ~2min as we still use the old ListView which is not really capable of handling such an amount.

We are working on it, but as this is not a regression of RC and needs a lot of changes, it will be not in 3.0.

@steve-s-71
Copy link
Author

@tobiasKaminsky I buy in you explanation in regards of photos - I have a lot there. But the same issue happens to the Favorites item - although I have only one (for testing).

@tobiasKaminsky
Copy link
Member

If you have only one favorite this is indeed strange.
How long did you wait?

Can you create us a test account and test if it happens also there and send the credentials to android at nextcloud dot com with a reference to this issue?

@xXSTrikeXx
Copy link
Contributor

I waited 5 mins for photos, but favorites are still Loading correctly.

@steve-s-71
Copy link
Author

steve-s-71 commented Jan 20, 2018

Hi @tobiasKaminsky

How long did you wait?

Approx. 10 Minutes.

can you create us a test account

I have done that. Under the test account I can immediately see the Photos that are stored by default and also the 2 Test Favorites.

Is it possible that the issue depends on the total number of files stored under an account, rather than on the number of files that are marked as Favorite? Under my productive Account I have ~30000 files.

@tobiasKaminsky
Copy link
Member

Is it possible that the issue depends on the total number of files stored under an account, rather than on the number of files that are marked as Favorite? Under my productive Account I have ~30000 files.

As you are our search expert, @icewind1991, can this make a difference?

@icewind1991
Copy link
Member

Does the same slowdown occur in the web interface, and if so please open an issue for this in the server repo

@steve-s-71
Copy link
Author

No @icewind1991, in the the web interface it works seamlessly

@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 3.1.0 milestone Feb 13, 2018
@tobiasKaminsky tobiasKaminsky self-assigned this Feb 13, 2018
@tobiasKaminsky
Copy link
Member

We are currently working on a fix, which then will be in 3.1

@tobiasKaminsky tobiasKaminsky removed this from the Nextcloud App 3.1.0 milestone Mar 15, 2018
@tobiasKaminsky
Copy link
Member

This requires some modification on server side, which will be in NC14, so we have to wait for them.

@asonix
Copy link

asonix commented Jul 5, 2019

Has there been any update on this? I feel like making individual queries for all potential photos on a nextcloud instance is probably the wrong way to go about this. I have many gigabytes of photos on my server running on low-power hardware. There's no way the photos section of the app will ever work for me with the current implementation.

In contrast, the web photos app works fine (with some lag), since it doesn't query every photo on the server, just photos in the current folder.

@tobiasKaminsky
Copy link
Member

There is now a PR ready to test (currently for photos only):
#2250 (comment)

@tobiasKaminsky
Copy link
Member

To centralize this please report anything found in #2250.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants