[Digikam-devel] [Bug 130883] New: overexposure indicator in color management is saved as part of the picture

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 130883] New: overexposure indicator in color management is saved as part of the picture

Erik van der Velden
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=130883         
           Summary: overexposure indicator in color management is saved as
                    part of the picture
           Product: digikam
           Version: unspecified
          Platform: Gentoo Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: digikam-devel kde org
        ReportedBy: erikvandervelden netscape net


Version:           9.0 svn (using KDE KDE 3.5.3)
Installed from:    Gentoo Packages
Compiler:          x86_64-pc-linux-gnu-3.4.6
OS:                Linux

When using the color management (fix->colors->color management) with the new 'over exposure' indicator, the over exposed part of the picture are hightlighted in nice bright colors.
However, when you click on OK, picture is saved as it is at that moment on the screen, i.e., with the overexposed parts still indicated by bright colors.

It can easily reproduced by using the lightness adjustments to brighten the figure considerably.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] [Bug 130883] overexposure indicator in color management is saved as part of the picture

Gilles Caulier
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=130883         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From caulier.gilles free fr  2006-07-16 09:02 -------
SVN commit 562893 by cgilles:

digiKam from trunk : Color Management image plugin : fix over exposure rule with final rendering

BUG: 130883



 M  +2 -1      NEWS  
 M  +2 -2      imageplugins/imageeffect_iccproof.cpp  


--- trunk/extragear/graphics/digikam/NEWS #562892:562893
 @ -166,6 +166,7  @
 103 ==> 126199 : Nikon D70 comments in jpegs are all shown as "charset="Ascii".
 104 ==> 130381 : Automatic colorbalance and camera color balance checkboxes swapped.
 105 ==> 117248 : Opening default app when camera inserted return KIOExec error.
-106 ==>
+106 ==> 130883 : Overexposure indicator in color management is saved as part of the picture
+107 ==>
 
 ----------------------------------------------------------------------------------------------------
--- trunk/extragear/graphics/digikam/imageplugins/imageeffect_iccproof.cpp #562892:562893
 @ -1018,8 +1018,7  @
             //-- Calculate and apply the curve on image after transformation -------------
         
             Digikam::DImg img2(w, h, sb, a, 0, false);
-            m_curves->curvesLutSetup(Digikam::ImageHistogram::AlphaChannel,
-                                     m_overExposureIndicatorBox->isChecked());
+            m_curves->curvesLutSetup(Digikam::ImageHistogram::AlphaChannel, false);
             m_curves->curvesLutProcess(img.bits(), img2.bits(), w, h);
         
             //-- Adjust contrast ---------------------------------------------------------
 @ -1124,6 +1123,7  @
         KMessageBox::error(this, i18n("Sorry, it seems there is no embedded profile"), i18n("Profile Error"));
         return;
     }
+
     QString intent;
     cmsHPROFILE selectedProfile = cmsOpenProfileFromMem(profile.data(), (DWORD)profile.size());
     QString profileName         = QString((cmsTakeProductName(selectedProfile)));
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel