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

TIFF: Support all features #1612

Closed
iwinstar opened this issue Oct 9, 2021 · 10 comments
Closed

TIFF: Support all features #1612

iwinstar opened this issue Oct 9, 2021 · 10 comments
Assignees
Labels
idea Feedback wanted / feature request released Available in the stable release

Comments

@iwinstar
Copy link

iwinstar commented Oct 9, 2021

version: 211007-8f55d6f8-Linux-x86_64 (latest)

detail logs:

INFO[2021-10-09T07:51:37Z] tiff: converting photo-1415886541506-6efc5e4b1786.tiff to jpg
ERRO[2021-10-09T07:51:38Z] resample: can't open photo-1415886541506-6efc5e4b1786.tiff
WARN[2021-10-09T07:51:38Z] index: failed converting photo-1415886541506-6efc5e4b1786.tiff to jpeg (tiff: invalid format: not enough pixel data)
INFO[2021-10-09T07:52:33Z] tiff: converting photo-1584886781099-0664a292d9b0.tiff to jpg
ERRO[2021-10-09T07:52:33Z] resample: can't open photo-1584886781099-0664a292d9b0.tiff
WARN[2021-10-09T07:52:33Z] index: failed converting photo-1584886781099-0664a292d9b0.tiff to jpeg (tiff: unsupported feature: color model)

"tiff: unsupported feature: color model" seems easy to fix:
photoview/photoview#242

"tiff: invalid format: not enough pixel data" no idea how to fix it.

I also provide those two photos for you guys to test. Thanks.

@iwinstar
Copy link
Author

iwinstar commented Oct 9, 2021

each photo is 40MB+, seems too large to directly upload under this issue, any other way?

Photos.zip
Photos.z01.zip
Photos.z02.zip
Photos.z03.zip

After download those files,

rename Photos.z01.zip to Photos.z01
rename Photos.z02.zip to Photos.z02
rename Photos.z03.zip to Photos.z03

Then decompress Photos.zip

@lastzero
Copy link
Member

lastzero commented Oct 9, 2021

I trust that some TIFF files can't be decoded... that's the full TIFF spec: https://www.adobe.io/content/dam/udp/en/open/standards/tiff/TIFF6.pdf

The Go authors didn't implement all of it. We don't have the financial resources either if Google doesn't.

@lastzero lastzero changed the title Resample Tiff photos error. TIFF: Support all features Oct 9, 2021
@lastzero lastzero added idea Feedback wanted / feature request unfunded labels Oct 9, 2021
@lastzero lastzero removed the unfunded label Nov 1, 2021
@maxammann
Copy link

We actually could use Darktable for conversion from TIFF to JPEG.

@iwinstar
Copy link
Author

iwinstar commented Dec 1, 2021

root@nas:/photoprism# darktable-cli originals/photo-1415886541506-6efc5e4b1786.tiff a.jpg
[rawspeed] (photo-1415886541506-6efc5e4b1786.tiff) rawspeed::RawImage rawspeed::RawDecoder::decodeRaw(), line 273: const rawspeed::TiffIFD* rawspeed::TiffIFD::getIFDWithTag(rawspeed::TiffTag, uint32_t) const, line 219: failed to find 1 ifs with tag 0x828e
[export_job] exported to `a.jpg'

root@nas:/photoprism# darktable-cli originals/photo-1584886781099-0664a292d9b0.tiff b.jpg
[export_job] exported to `b.jpg'

photo-1415886541506-6efc5e4b1786.tiff 45MB(tiff) -> 12MB(jpg)
photo-1584886781099-0664a292d9b0.tiff 42MB(tiff) -> 4.3MB(jpg)

It seems fine for me.

@maxammann
Copy link

raw0037.tif.xmp.zip

Nice, here is another example which works and has a corresponding xmp file.

@CDeLeon94
Copy link

What's the solution for now, manually convert via darktable-cli?
Does that just leave the originals un-index and un-imported, with only the conversions being present within photoprism?

@lastzero
Copy link
Member

The solution would be to have time to build one, which we don't have at the moment. This issue has not received much attention from users, so it has not been a top priority for us, given the number of other requests. A few lines of code should resolve it if you use an external converter instead of the built-in TIFF library (don't ask me why Google didn't implement all the TIFF features, maybe they ran out of money).

@lastzero
Copy link
Member

lastzero commented Oct 19, 2022

It would be very helpful to have file samples available for testing, see our contact page for details:

https://photoprism.app/contact

Edit: Sorry, just saw you already added a ZIP with samples! Thank you :)

@lastzero lastzero self-assigned this Feb 13, 2023
@lastzero
Copy link
Member

@iwinstar @CDeLeon94 @maxammann Since we added support for the ImageMagick "convert" command to support vector formats (see issue #2207), it is possible that previously unsupported TIFF files will work now.

The upcoming preview build 230213-32483cc88 can be used for testing. Your help is much appreciated!

@lastzero lastzero added the please-test Ready for acceptance test label Feb 13, 2023
lastzero added a commit that referenced this issue Feb 21, 2023
Signed-off-by: Michael Mayer <michael@photoprism.app>
lastzero added a commit that referenced this issue Feb 21, 2023
Signed-off-by: Michael Mayer <michael@photoprism.app>
@graciousgrey graciousgrey added tested Changes have been tested successfully and removed please-test Ready for acceptance test labels Apr 24, 2023
@graciousgrey graciousgrey added released Available in the stable release and removed tested Changes have been tested successfully labels May 3, 2023
@lastzero lastzero moved this to Released 🌈 in Roadmap 🚀✨ Jun 8, 2023
@lastzero
Copy link
Member

lastzero commented Nov 3, 2023

If you don't use our Docker image, please make sure to install the recommended dependencies including ImageMagick to ensure full file format support. See the "Dependencies" section of our package installation docs for more details:

https://dl.photoprism.app/pkg/linux/README.html

Thank you very much! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
idea Feedback wanted / feature request released Available in the stable release
Projects
Status: Release 🌈
Development

No branches or pull requests

5 participants