-
Notifications
You must be signed in to change notification settings - Fork 49
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 optional dependency on pyvips-binary
#507
base: master
Are you sure you want to change the base?
Conversation
Nice! Yes, maybe a separate repo is better. |
Allowing users to install pyvips with binary packages via: pip install "pyvips[binary]" See: https://pypi.org/project/pyvips-binary/.
ea769b2
to
d375529
Compare
pyvips-binary
packagepyvips-binary
... here are some testing notes: $ pkg-config --exists --print-errors vips
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
Package 'vips', required by 'virtual:world', not found
$ pip3 install pyvips
$ python -c "import pyvips; print(pyvips.API_mode)"
ModuleNotFoundError: No module named '_libvips'
During handling of the above exception, another exception occurred:
[...]
OSError: cannot load library 'libvips.so.42': libvips.so.42: cannot open shared object file: No such file or directory.
[...]
$ pip3 install pyvips-binary
$ python -c "import pyvips; print(pyvips.API_mode)"
True
$ python -c "import pyvips; pyvips.Image.black(1024, 1024).write_to_file('x.jpg')"
$ file x.jpg
x.jpg: JPEG image data, Exif standard: [TIFF image data, little-endian, direntries=6, orientation=upper-left, xresolution=86, yresolution=94, resolutionunit=2], baseline, precision 8, 1024x1024, components 1 |
Ah fantastic! That's so cool. I'll test here as well. Shall we put you on the |
Is is possible to invert the sense of this option? I'm sure most people should use the netvips binaries rather than the system ones. So |
Feel free to add me to the authors list! I usually use the same email address that's on my GitHub profile.
I had the same thought, but it doesn't seem to be possible at the moment. This future possible enhancement is discussed in detail here: https://discuss.python.org/t/4898. |
Maybe we could just make another pyvips package ( |
That approach might work, but I'm uncertain if it's the best option for users who prefer relying on system binaries. For example, Zulip tends to favor using libvips from the package manager (see e.g. commit zulip/zulip@e8f7e28). Additionally, pyvips is commonly used by users who process whole-slide images, something that the The Perhaps we should consider catching the Details
|
Allowing users to install pyvips with binary packages via:
$ pip install "pyvips[binary]"
See: https://pypi.org/project/pyvips-binary/.
Marked as draft due to this notes:
Perhaps this should be in a separate repository?Done: https://github.com/kleisauke/pyvips-binary-static-ffi
variant. The plan is to distribute the-static
variant when libvips 8.16 is released.pyvips/pyvips-binary/scripts/cibw_before_build.sh
Lines 14 to 16 in ea769b2
(this will also require a major bump in NetVips, see e.g. commit kleisauke/net-vips@8856756)