Bug 182272 - color profile and white balance
Summary: color profile and white balance
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: ColorManagement-Profiles (show other bugs)
Version: 0.10.0
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-29 10:25 UTC by Harald Hoyer
Modified: 2022-02-01 11:19 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 1.0.0
Sentry Crash Report:


Attachments
Patch to fix some icc problems (1.40 KB, patch)
2009-07-05 21:57 UTC, David Eriksson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Hoyer 2009-01-29 10:25:10 UTC
Version:           digikam-0.10.0-0.12.beta8.fc10.x86_64 (using KDE 4.1.4)
OS:                Linux
Installed from:    Fedora RPMs

It seems that only eog and gimp display the image with the correct white balance. 
http://www.harald-hoyer.de/gallery2/d/31063-1/IMG_1501-3.JPG

What is wrong here? Does digikam not honor the white point defined in the image color profile?
Comment 1 Harald Hoyer 2009-01-29 10:26:43 UTC
Might be the same with the images mentioned here:
http://www.harald-hoyer.de/personal/blog/color_management
Comment 2 Harald Hoyer 2009-02-11 12:35:35 UTC
another example for a wide gamut profile:

$ wget http://www.harald-hoyer.de/files/IMG_2387-wide-gamut.JPG
$ identify -verbose IMG_2387-wide-gamut.JPG
...
  Profiles:
    Profile-exif: 8412 bytes
    Profile-icc: 560 bytes
      Wide Gamut RGB
...

see the image with wrong colors
$ display IMG_2387-wide-gamut.JPG

now we convert it to sRGB
$ jpegicc IMG_2387-wide-gamut.JPG IMG_2387-sRGB.JPG

and we can see the image with the correct colors
$ display IMG_2387-sRGB.JPG

digikam does not even show that there is an embedded profile in the image browser nor does showFoto in the "convert to workspace color profile" dialog.
Comment 3 Corinna Vinschen 2009-02-16 14:16:54 UTC
This problem still appears in 0.10.0-rc1, AFAICS.  The same photo
viewed using Gimp looks visibly more warm than in Digikam's Editor,
when using the exact same Workspace and Monitor Color Profiles.
Only when switching off Color Management in both, Gimp and Digikam,
the photo loks identical between the two.
Comment 4 Marcel Wiesweg 2009-06-22 10:38:35 UTC
*** Bug 197353 has been marked as a duplicate of this bug. ***
Comment 5 David Eriksson 2009-07-05 21:57:14 UTC
I think part of the problem is that embedded profiles in jpeg are not read and that an embedded profile is not used as input profile when converting the image.
The attached patch fixes these problems, at least to the extent that the test image looks good. I don't know if useBuiltin should be tested before embedded or not.
Comment 6 David Eriksson 2009-07-05 21:57:59 UTC
Created attachment 35069 [details]
Patch to fix some icc problems
Comment 7 Marcel Wiesweg 2009-08-09 00:20:42 UTC
SVN commit 1009011 by mwiesweg:

Patch by David Eriksson:
Fix very subtle bug in the JPEG loader preventing embedded ICC profiles to be read because
the reading structure was destroyed before.
Awesome that you have found that bug!

CCBUG: 182272

 M  +1 -1      jpegloader.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1009011
Comment 8 caulier.gilles 2009-08-15 11:27:49 UTC
Harald,

Can you test patch applied by Marcel in current implementation from svn (1.0.0-beta4) ?

Thanks in advance

Gilles Caulier
Comment 9 Milan Knížek 2009-08-16 20:56:11 UTC
I have tested the current svn - embedded profiles in JPEGs are now recognised correctly by the editor (I have tested an image in the synthetic BRG.icc space so that misbehaviour is clearly visible).

One remaining thing is that digiKam still asks once, twice or even three-times about conversion/assignment/do nothing on ICC profile mismatch with the working space - I do not know if that is related or not. (I have not found what triggers the number of pop-up windows.)
Comment 10 Marcel Wiesweg 2009-08-16 22:22:09 UTC
Yes the code is in a sort of intermediate state currently there are some new bugs in it but I am preparing some more changes still. It's work in progress until the next beta.
Comment 11 Marcel Wiesweg 2009-08-20 23:22:25 UTC
The original bug is fixed. All proposed testcases here are displayed correctly on the image editor (img_2387 is not strikingly different, but I can see a difference) The problem of showing three dialogs is a different problem, I will fix that at the weekend.