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

Background thumbnail generation [$25] #18

Closed
8h2a opened this issue Feb 11, 2015 · 9 comments
Closed

Background thumbnail generation [$25] #18

8h2a opened this issue Feb 11, 2015 · 9 comments

Comments

@8h2a
Copy link

8h2a commented Feb 11, 2015

Feature request

User type: any

User level: any

Description

@TheDD
Since the app is extremely slow when opening a folder for the first time I think some actions like thumbnail generation should be moved to a background/cron job.

(I didn't review the code and have no idea if there's already some background work happening. If there is, I'd still like to discuss performance improvements)

@astroleo
I run owncloud 8.2.2 on a Raspberry Pi which works just fine (and thank you all for this!), but isn't terribly fast. When I share a link to a gallery it takes quite some time to browse through the gallery for the first visitor. opening the gallery, clicking on an image, then clicking the overlaid "next" arrow in the screen-sized image. I understand that opening one screen-sized image triggers the creation of the next one, but (for the Raspy) this isn't fast enough and the first user has to wait each time she clicks the "next" button.

My suggestion would be to add an option to create, right after uploading the album, the various required image sizes so that they are served quickly from the cache already for the first visitor.

Benefit / value

For OC installations on low-power computers (e.g. Raspberry Pi) the first visitor would already be able to smoothly scroll through an album.

Risk / caveats

If uploading a large amount of pictures, creating the cached versions may create quite some server load. The pre-caching task should therefore be given a low priority.

Sponsorship

@astroleo
I'm afraid I cannot sponsor anything yet, but I'd be happy to help developing / testing this new feature if an expert could provide me with some background on how caching works in OC.

Are you a developer willing to implement this feature?: @astroleo: yes

Can you sponsor the development of this feature or do you know somewone who can?: no

There is a $25 open bounty on this issue. Add to the bounty at Bountysource.

@oparoz
Copy link
Contributor

oparoz commented Feb 12, 2015

This will happen in 8.1, see this issue in core: owncloud/core#11685

@oparoz
Copy link
Contributor

oparoz commented Apr 10, 2015

Moved to 8.2...

@oparoz oparoz added the On hold label May 17, 2015
@oparoz oparoz added this to the backlog milestone Aug 31, 2015
@oparoz oparoz changed the title Background thumbnail generation Background thumbnail generation [$25] Sep 27, 2015
@oparoz oparoz added the bounty label Sep 27, 2015
@oparoz
Copy link
Contributor

oparoz commented Sep 27, 2015

owncloud/core#11685 has been moved to 9.0 :(

@oparoz
Copy link
Contributor

oparoz commented Sep 27, 2015

Depending on the timeline for the delivery of #221, I could add a method to the service to generate the thumbnail at the same time the information is collected.

@oparoz
Copy link
Contributor

oparoz commented Feb 28, 2016

@astroleo - I invite you to read owncloud/core#11685 where quite a few ideas are thrown around.

In short:

  • At upload time is the best solution as the server would have the encryption key
  • There may be a timing issue if it takes 5 minutes to generate a thumbnail. We can't make sync requests and we can't request the encryption key if files are encrypted
  • Create an app to manage the meta data stored in media files #550 is an app which manages images meta data. Maybe it could be modified to generate the thumbnail after analysis
  • There is already code to generate thumbnails offline. The idea is to test this on the WD Pi-drive to be able to tune it into an acceptable solution

@oparoz
Copy link
Contributor

oparoz commented Sep 4, 2016

This issue was moved to nextcloud/gallery#14

@Phil1988
Copy link

Phil1988 commented Mar 9, 2017

Its soo sad, that this one is dead.
The images/thumbnails makes the Owncloud almost unusable on my RPi 2...

@Vraana
Copy link

Vraana commented Mar 9, 2017

Sorry to say so, but this was the reason I switched to NextCloud....

@DeepDiver1975
Copy link
Member

Its soo sad, that this one is dead.

yes indeed - dividing the community by forking the project left many dangling pointers - on both side

just saying - no intent to start a flame war.

@oparoz oparoz removed the bounty label Apr 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants