-
Notifications
You must be signed in to change notification settings - Fork 78
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
(cont. g1408e42 r64) Writing JPEG previews fails #192
Comments
Looks like a bug: hdrmerge/src/DngFloatWriter.cpp Line 336 in 0412739
https://doc.qt.io/qt-5/qimagewriter.html#supportedImageFormats Should be "JPG" instead of "JPEG". "JPEG" was OK with Qt4: https://doc.qt.io/archives/qt-4.8/qimagewriter.html#supportedImageFormats |
"JPEG" was valid in Qt4 (alongside "JPG"): https://doc.qt.io/archives/qt-4.8/qimagewriter.html#supportedImageFormats But got dropped in Qt5: https://doc.qt.io/qt-5/qimagewriter.html#supportedImageFormats Fixes jcelaya#192
Hope for a new binary soon. |
I tried to patch the EXE with a hex editor, turning the first separate appearance of JPEG into JPG{#0} – it did not work as expected. |
Not fixed in HDRMerge_release-v0.6_continuous-71-gd7d8041_release_64 – please reopen. I converted the same Fujifilm raw file (DSCF4758.RAF) containing two frames, created by a Finepix S200 EXR in Dynamic Range mode. HDRMerge_master_continuous-64-g1408e42_release_64\hdrmerge.exe (2019-07-25, does not contain the string “JPG”) => DSCF4758-4758.dng (13056040 bytes, does not contain the string “JFIF”) HDRMerge_release-v0.6_continuous-71-gd7d8041_release_64\hdrmerge.exe (2020-02-24, contains the string “JPG” once) => DSCF4758-4758.dng (13056040 bytes, does not contain the string “JFIF”) Both DNG files are almost bit-exact the same (except for a creation timestamp in the header metadata). |
It's a packaging issue then. (I looked at the nightly build from here: https://keybase.pub/gaaned92/HDRMergeNightlyBuilds/) |
Success: I copied the qjpeg.dll from Luminance HDR to a subdirectory imageformats below the HDRMerge installation. It created a DNG file containign a JPEG preview. If you deploy it in new releases, you will use the current version of your Qt 5 installation. |
Thx for testing and the confirmation. |
Tested with v0.5. Consider reopening this issue: I get this error for all images of my batch. The DNG is valid, but the preview is missing.
|
It was solved in a nightly build. Stable version 0.5 is "ancient". |
There is a warning at the end of the output of the CLI version when trying to convert a dual capture Fujifilm EXR-DR RAF:
The message
Unsupported image format
makes me wonder if you just omitted a JPEG encoder library.Apart from that, I wonder if the current state might be a "nightly" release candidate. The last official release for Windows is still v0.5.
The text was updated successfully, but these errors were encountered: