:: Re: [Frei0r] Medians filter added
Top Page
Delete this message
Reply to this message
Author: Marko Cebokli
Date:  
To: Minimalistic plugin API for video effects
Subject: Re: [Frei0r] Medians filter added
I've made the suggested changes.

The new Kdenlive .xml is included

Marko Cebokli


On Thursday, February 07, 2013 08:50:43 AM Dan Dennedy wrote:
> On Thu, Feb 7, 2013 at 6:56 AM, Marko Cebokli <s57uuu@???> wrote:
> > On Wednesday, February 06, 2013 02:47:08 PM Dan Dennedy wrote:
> >> Please change the functions in small_medians.h to be static.
> >
> > OK, will do, no problem with that.
> >
> >> Also, please do not use an all numeric string parameter; that is abusing
> >> the rules. Either make a meaningful set of values like cross5, sq3x3,
> >> etc. or map them into the double [0,1] range.
> >
> > This is the problem of the missing integer/list parameter type in Frei0r,
> > we discussed this here some time ago.
> > The problem with divided [0,1] range is, that the number of choices cannot
> > be changed at a latter time, without breaking existing projects which
> > already use the plugin.
>
> agreed
>
> > Since I would like to leave open the possibility of adding new options in
> > the future, I choose the string type parameter.
> >
> > I can change the srings to be more descriptive than numbers, no problem.
> >
> > But I am not sure what would we gain with that? From the viewpoint of a
> > non- GUI user, he woud have to know the correct strings and type a few
> > more
> That is a very kdenlive-centric viewpoint because kdenlive provides
> its own UI XML for frei0r plugins. Other GUIs may only expose a text
> field and rely upon the parameter description to supply the set of
> possible values. I have made other string params put its permissible
> values in its description. Also, there are plenty of non-gui users of
> command line tools using melt, MLT XML, ffmpeg/avcon, gst-launch,
> script language bindings, and who knows what else. This makes their
> usage/authoring not only easier but also the reading of xml and
> scripts easier.
>
> > characters. So the strings would have to be written in the parameter
> > description. With number strings, adding ["0"..."10"] to the parameter
> > description would be sufficient.
> > Typing single digits is also simpler than typing fractions, as needed for
> > divided [0...1].
> >
> > And the GUI user won't see a difference anyway.
> >
> > Marko Cebokli
>
> _______________________________________________
> Frei0r mailing list - http://frei0r.dyne.org
> Free video plugins, minimal and cross-platform.
> http://mailinglists.dyne.org/cgi-bin/mailman/listinfo/frei0r