[Digikam-devel] [Bug 142443] red eye correction should change eye colour to an alternate colour

[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=142443         




------- Additional Comments From caulier.gilles gmail com  2007-03-06 21:12 -------
SVN commit 640090 by cgilles:

digiKam from trunk : digiKam 0.9.2 implementation continue here : red eyes correction again and again and again...

Now the tool have a dialog with a standard preview effect like other image editor tools. A screenshot can by seen at this url :

http://digikam3rdparty.free.fr/Screenshots/newredeyecorrectiontool.png

TODO : 

- make the Red Threshold setting more configurable using a slider to adjust finely the value.
- give another slider setting to adjust the Coloring Taint blending level.

TO STUDY : The content of the eye pupil is destroyed by the flash light reflected on the retinal. Why not use a real eye small pupil photo texture ? This tecture will be in black & white using an alpha channel. We will coloring it using the taint and merge it with the original image to remplace the red area... What do you think about ?

Other implementation patched with this commit : now the image editor plugin interface can render the preview in dialog using the current selection and not only using the full image data.

CCBUGS: 142443

 M  +262 -122  imageplugins/imageeffect_redeye.cpp  
 M  +83 -16    imageplugins/imageeffect_redeye.h  
 M  +17 -1     imageplugins/imageplugin_core.cpp  
 M  +3 -1      libs/widgets/imageplugins/imageguidewidget.cpp  
 M  +2 -1      libs/widgets/imageplugins/imageguidewidget.h  
 M  +3 -2      libs/widgets/imageplugins/imagewidget.cpp  
 M  +2 -1      libs/widgets/imageplugins/imagewidget.h  
 M  +33 -3     utilities/imageeditor/editor/imageiface.cpp  
 M  +10 -0     utilities/imageeditor/editor/imageiface.h