<div dir="ltr">Hi Tom and all,<br><br>After a quick look at the 4th patch, related to your optimized<br>Viterbi decoder, I have noticed that currently the convolutional<br>code definitions from the 'src/gsm/gsm0503_conv.c' are out of<br>the 'tests/conv' test coverage...<br><br>So, I would like to extend the test coverage. All I need are<br>the test vectors, which I'll add to existing ones. Some of them<br>I already found in your 4th patch, but some pending I need to<br>write myself.<br><br>Right now I have a simple question...<br>Let's look at one example:<br><br>{<br>    .name = "GSM RACH (non-recursive, flushed, not punctured)",<br>    .code = &gsm_conv_rach,<br>    .in_len  = 14,<br>    .out_len = 36, // ???<br>    .has_vec = 0,<br>    .vec_in  = { },<br>    .vec_out = { },<br>}<br><br>As I noticed, the 'in_len' may be taken from the code definition:<br><br>const struct osmo_conv_code gsm0503_rach = {<br>    .N = 2,<br>    .K = 5,<br>    .len = 14, // The 'in_len' is here!<br>    .next_output = xcch_output,<br>    .next_state = xcch_state,<br>};<br><br>But I don't know how to calculate the 'out_len'...<br>Could you please give me some hint?<br><br>I already started to work on your 4th patch:<br><a href="https://gerrit.osmocom.org/1542">https://gerrit.osmocom.org/1542</a><br><a href="https://gerrit.osmocom.org/1543">https://gerrit.osmocom.org/1543</a><br><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
</div>