[Kdenlive-devel] Fwd: Review Request 119095: GUI Added the vid.stab video stabilizer as a menu option.

[Steve Guilford]

Here's the project file entries for a source and a stabilized clip.

>   <kdenlive_producer audio_max="53" id="1" pix_fmt="yuv420p" 
> default_video="0" fps="59.9401" analysisdata="" name="GOPR0122.MP4" 
> videocodecid="h264" colorspace="709" videocodec="H.264 / AVC / MPEG-4 
> AVC / MPEG-4 part 10" 
> resource="/storage/goodinfo/vids_n_things/GOPR0122.MP4" 
> default_audio="1" audiocodec="AAC (Advanced Audio Coding)" 
> duration="984" file_hash="234baf14588497b9d40ca2fc1697850b" 
> aspect_ratio="1" channels="2" frequency="48000" video_max="0" 
> progressive="1" type="3" frame_size="1280x720" file_size="32802998">
>    <metaproperty tool="" 
> name="meta.attr.0.stream.creation_time">2009-03-22 01:31:12</metaproperty>
>    <metaproperty tool="" name="meta.attr.0.stream.handler_name">   
>  GoPro AVC</metaproperty>
>    <metaproperty tool="" 
> name="meta.attr.0.stream.language">eng</metaproperty>
>    <metaproperty tool="" 
> name="meta.attr.1.stream.creation_time">2009-03-22 01:31:12</metaproperty>
>    <metaproperty tool="" name="meta.attr.1.stream.handler_name">   
>  GoPro AAC</metaproperty>
>    <metaproperty tool="" 
> name="meta.attr.1.stream.language">eng</metaproperty>
>    <metaproperty tool="" 
> name="meta.attr.compatible_brands">avc1isom</metaproperty>
>    <metaproperty tool="" name="meta.attr.creation_time">2009-03-22 
> 01:31:12</metaproperty>
>    <metaproperty tool="" name="meta.attr.major_brand">avc1</metaproperty>
>    <metaproperty tool="" name="meta.attr.minor_version">0</metaproperty>
>   </kdenlive_producer>
>   <kdenlive_producer audio_max="53" duration="984" colorspace="709" 
> fps="59.9401" frame_size="1280x720" analysisdata="" 
> description=""Stabilised"" file_size="38981" 
> aspect_ratio="1" file_hash="fda51d56654936bd4c263d8e15427106" type="9" 
> id="2" name="GOPR0122.MP4.mlt" progressive="1" 
> resource="/storage/goodinfo/vids_n_things/GOPR0122.MP4.mlt">
>    <metaproperty tool="" 
> name="meta.attr.comment">"Stabilised"</metaproperty>
>   </kdenlive_producer>

I don't see any issues with removing the deprecated stabilizers.

Doing so will only take a few minutes.  I'll take care of it in a day or 
so...


On 07/08/2014 09:50 AM, Steinar H. Gunderson wrote:
> On Tue, Jul 08, 2014 at 11:34:40AM +0100, Evert Vorster wrote:
>> I would keep the other two stabilizers around for a while, there is no harm
>> in having a choice, is there?
> I keep seeing too many people think this way, and yes indeed, there is harm
> in having a choice. It is not a meaningful choice to users; how would they
> know which one to choose? And it means more code to be maintained, more
> clutter in the UI, and more confusion. The user does not want three different
> video stabilization filter, he/she just wants one that works. I would love if
> free software would be better at realizing that choice fundamentally tends to
> make people unhappy.
>
> The only real argument for keeping them that I can see would be that we
> shouldn't break existing projects without a good reason, but Dan has already
> pointed out that removing them won't.
>
> /* Steinar */

-- 
Steve Guilford
http://www.dbplugins.com