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

Add support for Imaginary #92

Open
enoch85 opened this issue Apr 2, 2023 · 10 comments
Open

Add support for Imaginary #92

enoch85 opened this issue Apr 2, 2023 · 10 comments

Comments

@enoch85
Copy link

enoch85 commented Apr 2, 2023

Nextcloud recently move towards Imaginary, and it would be great if this app could support that instead of Imagick.

nextcloud/server#13099
https://okxo.de/speed-up-nextcloud-preview-generation-with-imaginary/
https://github.com/nextcloud/vm/blob/master/apps/imaginary.sh

@ariselseng
Copy link
Owner

Imagick is optional and only used for embedded TIFF previews.

@enoch85
Copy link
Author

enoch85 commented Apr 2, 2023

Still, think that Imagick should be replaced with Imaginary since that's where Nextcloud is going in general. :)

@ariselseng
Copy link
Owner

@enoch85 With Imaginary, the previews is not cached in Nextcloud appdata. Is that correct?
I see that the default server/lib/private/Preview/Imaginary.php does not support tiff.

It would be awesome if Imaginary could support raw files (maybe it can via extensions ?). Then this app could even fallback to Imaginary for raw files without an embedded jpeg file. There is always files that does have a preview, and a fallback to a http service to actually convert the raw image to jpg would be so nice. Then this app would be complete and actually work for all raw files.
An idea I just got would be to setup a docker container, similar to imaginary, but with a simple http service that would call cli commands (or using some library) to convert the raw image to jpg and serve it.

@enoch85
Copy link
Author

enoch85 commented Aug 31, 2023

With Imaginary, the previews is not cached in Nextcloud appdata. Is that correct?

It is. All the previews that are generated are done so on the fly, but then saved in cache for later use.

An idea I just got would be to setup a docker container, similar to imaginary, but with a simple http service that would call cli commands (or using some library) to convert the raw image to jpg and serve it.

That sounds great!

Maybe even better would be to talk to the core team devs of Nextcloud and make it happen in the actual core? Also cc @szaimen on this.

@szaimen
Copy link

szaimen commented Aug 31, 2023

An idea I just got would be to setup a docker container, similar to imaginary, but with a simple http service that would call cli commands (or using some library) to convert the raw image to jpg and serve it.

That sounds great!

I am not sure about imaginary's capabilities but possibly it could also convert raw files into jpeg? Did not investigate yet. Depends on the supported image formats...

@ariselseng
Copy link
Owner

Maybe even better would be to talk to the core team devs of Nextcloud and make it happen in the actual core? Also cc @szaimen on this.

You mean having a provider that can connect to this http service, just like done for Imaginary in core?

@enoch85
Copy link
Author

enoch85 commented Aug 31, 2023

Maybe even better would be to talk to the core team devs of Nextcloud and make it happen in the actual core? Also cc @szaimen on this.

You mean having a provider that can connect to this http service, just like done for Imaginary in core?

Yeah something like that. Or would it possible to contribute to Imaginary to support RAW? Better to have one service than several, and since Imaginary now is the new "standard", maybe that would be the way forward?

@dewi-ny-je
Copy link

I found this ticket while looking for imagick in Nextcloud Snap, which does not provide it by default and therefore makes it impossible for me to use the related feature.
The reason why imagick is not provided anymore is that it has a terrible history of security, so replacing it completely or at least stopping using it should be considered:
nextcloud-snap/nextcloud-snap#882

@ariselseng
Copy link
Owner

@dewi-ny-je Most of this apps functionality works with gd and imagick is not a requirement.

@Forza-tng
Copy link

AFAIK, Imaginary does not support any raw formats such as DNG, CR2 or NEF. However, would it be possible to use this app as a filter that sends a tiff/PNG file to Imaginary for image preview generation?

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

No branches or pull requests

5 participants