Bug 226382

Summary: Curves Adjust dialog broken: No Histogram and no effect when changing the line
Product: [Applications] digikam Reporter: Simon <simon.eu>
Component: Plugin-Editor-CurvesAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles, tschenser
Priority: NOR    
Version: 1.2.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 1.2.0
Sentry Crash Report:
Attachments: broken histogram
build log when running make -j3

Description Simon 2010-02-11 21:09:15 UTC 
Version:           1.2.0 (rev.: 1085904) (using 4.3.4 (KDE 4.3.4), Debian packages)
Compiler:          cc
OS:                Linux (i686) release 2.6.32-7.slh.7-sidux-686

r 1085904

Curves dialog is broken. See attached screenshot.
Comment 1 Simon 2010-02-11 21:09:43 UTC 
Created attachment 40692 [details]
broken histogram
Comment 2 Simon 2010-02-11 21:10:27 UTC 
(Notice that the target is being shown. The modified curve does not have any effect.)
Comment 3 Simon 2010-02-11 21:15:40 UTC 
Additional note: Loaded .grad files do work, but show a straight line as curve only.
Comment 4 Jens Mueller 2010-02-13 14:29:57 UTC 
Do the problem with current trunk still exist for you? I also noticed that, but as of today curves adjust work like expected.

Regards, Jens
Comment 5 caulier.gilles 2010-02-13 14:37:22 UTC 
Jens, 

I don't have changed something yet in this tool about preview and histogram computation. It's planed of course following #163286 entry.

Gilles Caulier
Comment 6 Jens Mueller 2010-02-13 15:15:43 UTC 
No Gilles, you dont change something there. But as there was (for me was) a problem, I want to know where it came from.
Comment 7 Simon 2010-02-13 15:17:08 UTC 
Created attachment 40740 [details]
build log when running make -j3

Still cannot compile. This time it is probably related to: 

/data/cworkspace/graphics$ svn up
svn: Unbekannter Hostname »anonsvn.kde.org«

This is the first time I see the unknown hostname message.
Comment 8 Jens Mueller 2010-02-13 15:31:52 UTC 
@Simon: Did make a clean build? Looks like some directories are not found and there where some moved in svn lately.

@Gilles: I also have a patch ready making all the public members of histogramwidget and curves widget private and use get and set functions instead. But I want to see all histogram related bugs gone before merge.

Regards, Jens
Comment 9 Simon 2010-02-21 22:36:29 UTC 
Finally able to build again.

Problem fixed in current SVN. Thanks.
Comment 10 Jens Mueller 2010-02-23 18:06:49 UTC 
Closing this entry.