Re: [Mlt-devel] [Kdenlive-devel] project profile matching

[Dan D. <da...@de...>]

On Mon, Sep 20, 2010 at 7:11 AM, jb <jb...@kd...> wrote:
> On Monday 20 September 2010 05:37:07 Dan Dennedy wrote:
>> On Sun, Sep 19, 2010 at 6:42 PM, Dan Dennedy <da...@de...> wrote:
>> > I noticed some recent commits to suggest a matching profile for the
>> > first clip added.
>> > I just noticed that it does not appear to checking aspect ratio.
>> > Therefore, for DV or DVD and similar SD sources, it does not pickup
>> > anamorphic widescreen. I tried to take a look at commit 4907 to see
>> > what you are doing, but, man, that is so littered with unrelated
>> > changes I lacked the patience to look through it.
>
> Yep, sorry I just upgraded my system and astyle 1.24 changed some default
> indentation settings which messed up the commit. I just updated the profile
> detection so that it now uses the aspect ratio and shows a list of matching
> profiles to select.
>
> Currently, that detection is only done for the very first clip added to the
> project, and I just added an entry in the "Project" menu to find profile from
> currently selected clip.
>
>> Not yet merged, but here is more info. These new properties reflect
>> MLT's "cooked" values that represent the detected media properties by
>> taking into consideration heuristics between the muxer and codec
>> values, selected stream indexes, and overrides (e.g. force_fps):
>>
>> meta.media.sample_aspect_num=1
>> meta.media.sample_aspect_den=1
>> meta.media.frame_rate_num=30
>> meta.media.frame_rate_den=1
>> meta.media.width=1280
>> meta.media.height=720
>> meta.media.top_field_first=0
>> meta.media.progressive=1
>
> Ok, I will see if I can make use of it when it is committed.

One more thing about this. When merged, these will deprecate
aspect_ratio, source_fps, real_width, and real_height. Of course, they
will not go away immediately, but I will convert all MLT's code over
to using the new properties. It just means, as far as maintenance is
concerned, no attention will be paid to the deprecated properties.

-- 
+-DRD-+