:: Re: [Frei0r] Fx parameters / Defaul…
Página Inicial
Delete this message
Reply to this message
Autor: d.j.a.y
Data:  
Para: Minimalistic plugin API for video effects
Assunto: Re: [Frei0r] Fx parameters / Default value ?
Yop,

Dan thanx for clarifying this out...
I will soon implement (2) and pull.

See'ya.

----- Mail original -----

De: "Dan Dennedy" dennedy.org>
À: "Minimalistic plugin API for video effects" lists.dyne.org>
Envoyé: Dimanche 1 Mai 2016 18:39:14
Objet: Re: [Frei0r] Fx parameters / Default value ?




On Sun, May 1, 2016 at 12:17 AM < d.j.a.y@??? > wrote:


Hi there,

I am about to propose a patch for the "distort0r" effect
adding a new 'velocity' parameter controlling ... the speed
of the sinus wave.

https://github.com/d-j-a-y/frei0r/commit/f122c70b94c47395d8a4c2eeee3ae699c3093a5a

Without a default value set, there is a kind of "visual





The default value in your patch is 1.0, and we require params to have default values now.

<blockquote>
compatibility" break : new version of the effect is slower,
to reach the current (v1.5) visual effect of distort0r
(and no adding "visual compatibility" break) the new parameter
should be set to 80% (0.8).

blockquote>


<blockquote>

1) Any idea about the default parameter value ?

blockquote>



According to a strict reading of this description, set it to 0.8 to preserve the current behavior. You should strive not to break people's existing projects and scripts.

<blockquote>

Or maybe i can add a second (bool) parameter :
False - old behavior (based on time)
True - new behavior (based on velocity) .

2) What do you think/prefer?


blockquote>



I think the bigger compatibility issue is that your f0r_update now ignores the "time" call argument. Perhaps you should add a parameter to choose the old versus new behavior (default to old). Then, you can have a more preferable or suggested default for Velocity.


_______________________________________________
Frei0r mailing list - http://frei0r.dyne.org
Free video plugins, minimal and cross-platform.
https://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r



--

--

--