Bug 375338

Summary: git master: alpha shapes effect resets its parameters to 1000
Product: [Applications] kdenlive Reporter: Wegwerf <wegwerf-1-2-3>
Component: Video Effects & TransitionsAssignee: Vincent PINON <vpinon>
Status: RESOLVED FIXED    
Severity: normal CC: french.ebook.lover, snd.noise
Priority: NOR    
Version First Reported In: git-master   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Wegwerf 2017-01-20 14:04:59 UTC
With recent git master and MLT, when trying to use the alpha shapes effect, it always (re)sets all its parameters to 1000, thus becoming useless. A reset works until the user tries to change a parameter, either using the sliders, or by changing one of the input field values. While the effect UI seems to be working, when you click away, then select the effect again, all parameters are reset to 1000.
Comment 1 farid 2017-02-05 20:12:19 UTC
From my tests I can't reproduce... could you maybe provide a step by step instructions? 

I am using MLT 6.5 and Kdenlive 17.03.70.
Comment 2 Wegwerf 2017-02-05 22:05:43 UTC
Will retest, there should have been a commit I saw.
Comment 3 Wegwerf 2017-02-07 10:26:41 UTC
Retested: the resetting of parameters is gone, but most users will now by the problem that Farid also hit: all parameters default to 0, which aren't the correct defaults.

Position X, Y, as well as Size X, Y, and in particular Max should have default values other than 0. Max should be 1000, meaning full alpha.

So this bug is only half fixed, but default values of 0 are incorrect and make the effect unusable to most users.
Comment 4 Wegwerf 2017-02-07 10:27:46 UTC
At this time, I can work around in my projects using the Rectangular Alpha mask effect. But the defaults should be fixed properly.
Comment 5 alcinos 2017-03-06 14:20:37 UTC
Current master, I have the following defaults:
Position X: 500
Position Y: 500
Size X: 100
Size Y: 100
Tilt: 500
Transition width: 200
Min: 0
Max: 1000

Do you confirm it is fixed?
Comment 6 Wegwerf 2017-03-06 19:09:22 UTC
Can you please nudge the Kdenlive master ppa for a new build, so I can test? Thank you very much!
Comment 7 Wegwerf 2017-03-13 19:16:47 UTC
Confirming that this bug is fixed. Thank you very much! Moving BR to resolved-fixed.