I don't mind you dropping support for older IM versions but I'm not sure how Debian/Ubuntu users will feel about this change.
Luckily, I'm the Debian maintainer of img2pdf so I can adjust things…
Which commits are you refering to?
Currently, with IM7, the tests pass fine for me locally. Unfortunately, this breaks the Travis tests because those still use Ubuntu which only has IM6 available.…
Agreed. pdfrw should go.
Related to the tests, I have a local commit that switches the testsuite to imagemagick 7. Would that be a problem for Fedora?
Also, do you currently build img2pdf with…
6eec05c11c
workaround for Pillow >= 8.3.0 which limits CCITT Group4 strip size to 64 KB (closes: #122)
@sbraz do you have imagemagick 7 on gentoo? I just installed 7.1.0.5 and adapted the tests so that they work with IM7. I'm pondering to drop IM6 support because of all the differences in the output…
I think
first_frame_onlydoesn't help @johannesnussbaum because apparently his desired image is on the second frame.
I do not see an attached image, so I cannot check. But @johannesnussbaum…
From the --help output:
--first-frame-only By default, img2pdf will convert multi-frame images
like multi-page TIFF or animated GIF images to one
…I reported this as https://github.com/python-pillow/Pillow/issues/5740
Alternatively, if anybody knows of another group4 encoder, I'm all ears.
It might be possible to just add multi-strip support to
ccitt_payload_location_from_pilbut a better solution would be to drop that hack altogether and do a CCITT Group4 encoding in a different…
Ouch! Thanks a lot for bringing up this problem.
It might be possible to just add multi-strip support to ccitt_payload_location_from_pil but a better solution would be to drop that hack…