<div dir="ltr"><div><div><div>Hi there. <br><br></div>Tested, and the stabilizer works as advertised. <br><br></div>I would keep the other two stabilizers around for a while, there is no harm in having a choice, is there? I suppose we could mark them as depreciated. <br>
<br></div>-Evert Vorster-<br><div><br><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 8 July 2014 08:32, Vincent PINON <span dir="ltr"><<a href="mailto:vincent.pinon@laposte.net" target="_blank">vincent.pinon@laposte.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Hello,<br>
<br>
Sorry for the late reaction;<br>
Patch applied to activate vid.stab, on git master (stable) branch.<br>
<br>
Please test and comment ; do we need to keep older stab filters ?
(for projects compatibility)<br>
<br>
BR,<br>
<br>
Vincent<br>
<div><br>
<br>
-------- Message original --------
<table cellpadding="0" cellspacing="0" border="0">
<tbody>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Sujet: </th>
<td>Review Request 119095: GUI Added the vid.stab video
stabilizer as a menu option.</td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Date : </th>
<td>Thu, 03 Jul 2014 21:48:13 -0000</td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">De : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Répondre
à : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Pour : </th>
<td>Vincent Pinon <a href="mailto:vincent.pinon@laposte.net" target="_blank"><vincent.pinon@laposte.net></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Copie
à : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
</tbody>
</table>
<br>
<br>
<div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">
<table style="border:1px #c9c399 solid" cellpadding="8" bgcolor="#f9f3c9" width="100%">
<tbody>
<tr>
<td> This is an automatically generated e-mail. To reply,
visit: <a href="https://git.reviewboard.kde.org/r/119095/" target="_blank">https://git.reviewboard.kde.org/r/119095/</a>
</td>
</tr>
</tbody>
</table>
<br>
<table style="background-image:url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png');background-repeat:repeat-x;border:1px black solid" cellpadding="8" cellspacing="0" bgcolor="#fefadf" width="100%">
<tbody>
<tr>
<td>
<div>Review request for Vincent Pinon.</div>
<div>By Steve Guilford.</div>
<div style="margin-top:1.5em"> <b style="color:#575012;font-size:10pt">Repository: </b>
kdenlive
</div>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
<table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" bgcolor="#ffffff" width="100%">
<tbody>
<tr>
<td>
<pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">GUI Added menu option for new video stabilizer "vid.stab". Duplicated screen for videostab2 stabilizer for use with vid.stab.</pre>
</td>
</tr>
</tbody>
</table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" cellpadding="10" cellspacing="0" bgcolor="#ffffff" width="100%">
<tbody>
<tr>
<td>
<pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">Tested w/ current dev branch of Kdenlive.</pre>
</td>
</tr>
</tbody>
</table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">
<li>src/project/clipstabilize.cpp <span style="color:grey">(3085f31)</span></li>
<li>src/mainwindow.cpp <span style="color:grey">(ba54164)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119095/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>
</td>
</tr>
</tbody>
</table>
</div>
<br>
</div>
<br>
</div>
<br>------------------------------------------------------------------------------<br>
Open source business process management suite built on Java and Eclipse<br>
Turn processes into business applications with Bonita BPM Community Edition<br>
Quickly connect people, data, and systems into organized workflows<br>
Winner of BOSSIE, CODIE, OW2 and Gartner awards<br>
<a href="http://p.sf.net/sfu/Bonitasoft" target="_blank">http://p.sf.net/sfu/Bonitasoft</a><br>_______________________________________________<br>
Kdenlive-devel mailing list<br>
<a href="mailto:Kdenlive-devel@lists.sourceforge.net">Kdenlive-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/kdenlive-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/kdenlive-devel</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr">Evert Vorster<br>Chief Observer<br>WG Cook<br></div>
</div>