Bug 375338 - git master: alpha shapes effect resets its parameters to 1000
Summary: git master: alpha shapes effect resets its parameters to 1000
Status: RESOLVED FIXED
Alias: None
Product: kdenlive
Classification: Applications
Component: Video Effects & Transitions (other bugs)
Version First Reported In: git-master
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Vincent PINON
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-20 14:04 UTC by Wegwerf
Modified: 2017-03-13 19:16 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.