previewgenerator icon indicating copy to clipboard operation
previewgenerator copied to clipboard

rework to support multiple concurrent instances

Open nachoparker opened this issue 5 years ago • 13 comments

See https://github.com/nachoparker/previewgenerator/pull/1

Signed-off-by: nachoparker [email protected]

nachoparker avatar Jun 29 '19 21:06 nachoparker

@nachoparker Thank you so much for your PoC and your branch merge request. Considering your results, your tweaks would improve vastly the processing times in the Preview Generator app. I hope it gets merged anytime soon :)

jkingcis avatar Jul 03 '19 09:07 jkingcis

@nachoparker Awesome, I just found my cron job running for 35 hours now after uploading my whole image gallery, using only a single core, and already made some research about how PHP can use multiple cores/threads.

Well here an implementation has already been coded, many thanks for this!

MichaIng avatar Aug 08 '19 11:08 MichaIng

@rullzer The travis-ci build job fails because nextcloud requires atleast PHP 7.1, does this need to be fixed (aka PHP 7.0 removed from the pipeline)? Just a curious newb question. ;)

FleischKarussel avatar Sep 13 '19 07:09 FleischKarussel

@nachoparker just wondering if this is still alive - is there a todo you have, unfinished business?

jospoortvliet avatar May 06 '20 08:05 jospoortvliet

@jospoortvliet last time I checked we are waiting for feedback from the maintainer confirming we want this.

Since this has not happened I am shipping NCP with my own fork of this app for the time being until that happens.

nachoparker avatar May 06 '20 18:05 nachoparker

@jospoortvliet last time I checked we are waiting for feedback from the maintainer confirming we want this.

Since this has not happened I am shipping NCP with my own fork of this app for the time being until that happens.

Ok, so we're waiting for @rullzer - fair enough.

jospoortvliet avatar May 13 '20 07:05 jospoortvliet

@jospoortvliet last time I checked we are waiting for feedback from the maintainer confirming we want this. Since this has not happened I am shipping NCP with my own fork of this app for the time being until that happens.

Ok, so we're waiting for @rullzer - fair enough.

@jospoortvliet @nachoparker It looks like @rullzer has abandoned this project?

aniqueta avatar Aug 31 '20 03:08 aniqueta

Since there was just a commit, obviously not 🙂: https://github.com/rullzer/previewgenerator/commit/03665bc120ddd52aa334461b7f271f025b6fc449

MichaIng avatar Aug 31 '20 11:08 MichaIng

Since there was just a commit, obviously not 🙂: 03665bc

Awesome. The commit was just a few hour after my post. It had been dormant for a while. Does not seem like this support for multiple concurrent instances is coming in v3 (yet) though?

aniqueta avatar Aug 31 '20 21:08 aniqueta

@stefaang on Sep 18, 2019 •

I've been working on a python version of the preview generator with jpegtran for blazing fast image processing.

This snippet allows running multiple python processes.

I'll let you know when it's ready to be published :)

Hi, any progress? I couldn't apply @nachoparker's patches to current release :(

mailinglists35 avatar Sep 19 '20 23:09 mailinglists35

Hi, kindly asking for progress :-)

bmarwell avatar Dec 19 '20 19:12 bmarwell

@stefaang or @nachoparker , is there any progress on this? The branch maintained by @nachoparker doesn't seem to be updated, and there aren't any repos by @stefaang that is remotely related to nextcloud unfortunately.

hahagu avatar Mar 01 '21 09:03 hahagu

Hey @nachoparker or any-one else, is there any status update on this after 2 years? We've gone through the pandemic and I feel like it has gone through faster than my preview images.

StarSmasher44 avatar May 29 '21 13:05 StarSmasher44