<div dir="ltr"><br><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div><div><span style="color:rgb(7,55,99)"><font><span style="font-family:arial,helvetica,sans-serif"><span></span><a></a><span></span>Greeshma </span></font></span></div></div></div><div><span style="font-family:courier new,monospace"><br></span></div></div></div></div>
<br><div class="gmail_quote">On Tue, Mar 24, 2015 at 8:06 PM, Devin Heitmueller <span dir="ltr"><<a href="mailto:dheitmueller@kernellabs.com" target="_blank">dheitmueller@kernellabs.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Greeshma,<br>
<br>
It's good to see somebody working on EIA-708 support.  A few questions:<br>
<br></blockquote><div>Hello,</div><div><br></div><div>I actually started working on this 6-7 days back. This is part of qualification task for Outreach program for women'15.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- It looks like this patch just creates some basic data structures and<br>
enums.  Do you have additional patches which demonstrate this<br>
functionality?  I would be hesitant to see anything committed that<br>
doesn't actually result in a functional change in the code.  I think<br>
it would be better to see the development happen on some branch, and<br>
then the entire patch series can be submitted for upstream inclusion<br>
once you have something that works.  The alternative tends to result<br>
in code that needs to be heavily refactored as you make it actually<br>
work, dead code that isn't actually needed in the final solution, and<br>
introducing new constraints on existing functionality.<br>
<br></blockquote><div>As today is the deadline for application i have submitted all i could do till now. I am right now working on the proper functionalities.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- I believe we're currently only feeding the EIA-608 bytes to the cc<br>
decoder currently.  If we wanted to support EIA-708, the MPEG2 demux<br>
would need to be modified to create additional decoder instances and<br>
feed those bytes instead.<br></blockquote><div><br></div><div>Yes, MPEG2 demux has to be modified/extra features are to be added.As per my knowledge we now send captions included in picture user data.We need to change the decoder accordingly.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- We need to think about the GUI implications of supporting both<br>
EIA-608 and 708.  Do we now show eight menu options for CC1-4 for<br>
EIA-608 and CC1-4 for EIA-708?  Do we automatically pick EIA-708 over<br>
EIA-608 (or the reverse) if both are present?  Do we have some global<br>
configuration option which dictates which decoder gets used?<br></blockquote><div><br></div><div>EIA 708 has way lot more features than EIA 608.608 is kind of outdated format.There  is a way to convert 608 format to 708 but vice-versa is not possible.The transmission options decide which decoder to be used- eia608 uses line 21 transmission whereas eia708 is embedded in MPEG-2 streams. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- The window anchor support we could potentially do without having to<br>
support multiple subregions, given the current EIA-608 decoder always<br>
assumes a 32x16 character window.  The anchor support could be simply<br>
emulated by properly positioning the cursor within that window.<br>
<br>
- Adding support for some of these features could exacerbate known<br>
performance issues in the existing SPU renderer.  There are some<br>
assumptions made with regards to things such as font size and<br>
positioning which could cause a significant reduction in compositing<br>
performance if we provide more flexibility as allowed by EIA-708.  I<br>
can get into more specifics on this in a separate email.<br></blockquote><div><br></div><div>Thank you for the guidance.I am trying to contribute to it through Outreach Program for Women -OPW'15. </div><div>Even if its not possible i would definitely continue working on this and finish the decoder.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
On Tue, Mar 24, 2015 at 6:14 AM, greeshma <<a href="mailto:greeshmabalabadra@gmail.com">greeshmabalabadra@gmail.com</a>> wrote:<br>
><br>
> From 6260d6377da95f0dcc0e92e6342c4f9c540cc145 Mon Sep 17 00:00:00 2001<br>
> From: greeshmab <<a href="mailto:greeshmabalabadra@gmail.com">greeshmabalabadra@gmail.com</a>><br>
> Date: Tue, 24 Mar 2015 10:27:46 +0530<br>
> Subject: [PATCH] Added basic features of CIA-708 from wiki page<br>
><br>
> Signed-off-by: greeshma <<a href="mailto:greeshmabalabadra@gmail.com">greeshmabalabadra@gmail.com</a>><br>
> ---<br>
>  modules/codec/cc.c | 250<br>
> +++++++++++++++++++++++++++++++++++++++++++++++++++++<br>
>  1 file changed, 250 insertions(+)<br>
><br>
> diff --git a/modules/codec/cc.c b/modules/codec/cc.c<br>
> index 91e35ee..89754f6 100644<br>
> --- a/modules/codec/cc.c<br>
> +++ b/modules/codec/cc.c<br>
> @@ -1132,3 +1132,253 @@ static char *Eia608Text( eia608_t *h, bool b_html )<br>
>          Eia608Strlcat( psz, "</text>", i_size );<br>
>      return psz;<br>
>  }<br>
> +<br>
> +<br>
> +//Based on the Eia708 Wikipedia page<br>
> +//For SetPenAttributes function, Code is based on CCExtractor code<br>
<br>
</span>This could be a problem.  The current code for cc.c is LGPL, and using<br>
code from CCExtractor will result in the module having to be GPL<br>
licensed.<br>
<span class="HOEnZb"><font color="#888888"><br>
Devin<br>
<br>
<br>
--<br>
Devin J. Heitmueller - Kernel Labs<br>
<a href="http://www.kernellabs.com" target="_blank">http://www.kernellabs.com</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br></div></div>