Hi,<br><br>Attached two new patches to this email. <br><br>Patch 1 - Dirac encoding using Schroedinger library<br>Patch 2 - Using Schro encoder settings utilities to fetch and set encoding options rather than <br>              hardcoding the names in the Schro plugin module.<br>
<br><br>I am off on holiday from Friday to  the end of the Jan. I can make any minor changes suggested before Fri. <br><br>Regards,<br>Anuradha <br><br><div class="gmail_quote">On Mon, Dec 20, 2010 at 10:42 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hello,<br>
<br>
On Mon, Dec 20, 2010 at 10:13:50AM +1100, Anuradha Suraparaju wrote :<br>
<div class="im">> There are many Schro encoder options that an end user will probably never<br>
> ever use. So the number of options will be really long. In order to set an<br>
> option as advanced, we might probably have to parse the option names to<br>
> determine whether it is is an advanced option e.g. all the magic_* encoder<br>
> options.<br>
</div>OK.<br></blockquote><div><br>Implemented in Patch 2. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
> - "Schro has a mechanism for passing PTS through the encoder, with<br>
> > schro_encoder_push_full()<br>
> >   so StorePicturePTS() and friends is not useful."<br>
> ><br>
> will do this. I already used this call in another piece of software I wrote<br>
> ages ago. I was trying to keep the Schro plugin as similar to the dirac<br>
> plugin as possible.<br>
<br>
</div>Ok. But I think that even if the dirac similarity is nice, we should<br>
always try the best way, especially since the dirac plugin will die.<br>
<div class="im"><br>
> > - "schro_video_format_set_std_video_format() should be called<br>
> >  with SCHRO_VIDEO_FORMAT_CUSTOM rather than guessing a value -- CUSTOM<br>
> >  causes the encoder to guess a better value<br>
> ><br>
> As far as I know this is not true. CUSTOM means just that - the user needs<br>
> to customise the different video format parameters. Schro encoder does not<br>
> customise the encoder parameters if the video format is set to CUSTOM.<br>
<br>
</div>Well, that was told by some schroedinger developers, so well...<br>
<div class="im"><br>
<br>
> - "Should call schro_encoder_set_packet_assembly (schro_enc->encoder, TRUE);<br>
> > instead of<br>
> >   assembling packets manually"<br>
> ><br>
> ><br>
> Will do this. Again, I used this call in another plugin I wrote.<br>
</div>ok.<br>
<div class="im"><br></div></blockquote><div><br>Implemented in Patch 1.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">

> I guess it will be best if I send two patches. One patch covering the PTS<br>
> and assembling packets part with the options parsing code the same as it is<br>
> in the patch I sent earlier.<br>
><br>
> The second patch will contain the changes to the options parsing code using<br>
> the schro encoder options setting API.<br>
><br>
> How does this sound?<br>
</div>It seems the first patch is needed. It should probably be merged with<br>
the main one.<br>
<br>
I have no opinion about the necessity of the second one.<br>
<div><div></div><div class="h5"><br>
Best Regards,<br>
<br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a><br>
+33 672 704 734<br>
</div></div></blockquote></div><br>