[Digikam-devel] [Bug 148540] Improvements of adding noise tool

[Julien Narboux]

https://bugs.kde.org/show_bug.cgi?id=148540





--- Comment #59 from Julien Narboux <Julien narboux fr>  2010-03-25 13:46:26 ---
Gilles,

Sorry I am very busy these days.

I just tested rev 1107345, you did a wonderful job. The chroma noise looks very
realistic. The photographic distribution looks very nice as well. Maybe it
should just be the default.

I still see two problems:
- the grain size creates mosaics
- the photographic distribution option creates a color change, it seems that
the randomization always adds a value.

I don't understand why we can not create the noise in a separate layer starting
from midgrey and then merge it with the picture. To prevent the luminosity from
changing I think we juste need to add the second layer-128.

Concerning the check boxes, I think it would be better to have another kind of
expander widget with the check box directly included in the title of the
expander right after the arrow. This would factorize code about hiding/showing
the setings widgets and this widget could be used in the local contrast plugin
as well. 

Julien

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.