-
Notifications
You must be signed in to change notification settings - Fork 140
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 libvips #365
Comments
I can't take your request for instance but i keep it in the enchancement list for instance. PS: Personally, I looked for several options for image comparison and fell back on OpenCV through a python wrapper (not perfect for maintainability but very efficient : the only complexity is to save the image in filesystem, so i got some delay related and i had to implement lof of cache feature to limit that). I'm very curious about libvips capabilities. |
The biggest problem (which is also totally not your fault) is that either way installation through apt-get is relatively slow because we always need to update(the indexes) first, and then there's also extra package dependencies. I've also had trouble in the past with apt-get in pipelines - from time to time their update services breaks (for a whole day, for example), causing us to be stuck until it's fixed. In cases like this, the best solution is either a pre-built image or custom docker image.
I was considering taking that approach by building a native php library in rust connecting to OpenCV. On the other hand, libvips was easier to set up: installing libvips, installing composer package, ensuring php has ffi enabled, done. That's just my personal experience.
Fine with me! 👍 |
I'm using libvips inside PHP over ext-ffi. The docker image already supports enabling FFI, however of course libvips is missing.
At the moment I'm enabling it with the following inside my docker-compose file:
..but it would be nicer to have it by default in the 'fat' image.
For some context on the whole story, check https://stackoverflow.com/q/76416455/314056.
The text was updated successfully, but these errors were encountered: