[Cin] Fwd: Chromakey + blur testcase from IgorV
Igor BEGHETTO
igorbeg at visi1.org
Mon Feb 26 15:58:08 CET 2024
No, I would not change the code. I would left as it is. The reasons are
multiple, for me:
- if you open an old project, that project works right with A_KEY=1. An
user could have written, "it doesn't work", at that time. The test case
shows why it doesn't work.
- Now, you can change A_KEY value from 0 to 1, if needed, using the
"Keyframe parameters" window. If you change the code, that will no
longer be possible.
IgorBeg
26/02/2024 14:34, Andrew Randrianasulu wrote:
> For now we can try and modify in plugins/blur/blur.C
>
> void BlurMain::save_data(KeyFrame *keyframe)
> output.tag.set_property("A_KEY", config.a_key);
> replace config.a_key with 0 here ?
> so it will be always saved as 0
> void BlurMain::read_data(KeyFrame *keyframe)
> config.a_key = input.tag.get_property("A_KEY", config.a_key);
> with config.a_key = 0;
> so it will ignore saved 1 setting.
> At least this is my theory/idea for now.
> If we go with this plan we probably should left original lines
> commented out with "//" and add line saying why reading/writing forced
> to 0 for this param (due to disabled gui config in blurwindow.C)
More information about the Cin
mailing list