<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">(4 comments)</p><p style="white-space: pre-wrap; word-wrap: break-word;">Would be also nice to have a more universal solution for all kinds of measurements (+ToA256, +C/I), not only RSSI.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Do you think doing this in this patch makes sense? I think it would not be difficult to add this. It would be just the same as with the RSSI.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035/1/src/osmo-bts-trx/scheduler_trx.c">File src/osmo-bts-trx/scheduler_trx.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035/1/src/osmo-bts-trx/scheduler_trx.c@1281">Patch Set #1, Line 1281:</a> <code style="font-family:monospace,monospace">RSSI_FR_LOWER_FOUR</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why only four bursts? A regular TCH/F or FACCH/F frame is interleaved over 8 bursts, so you need to  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The problem is that picking the measurement values becomes a lot more difficult when I compute the average over all 8 samples. The problem is with the SID frames, then suddenly only half of the bursts are sent and recognizing this turned out to be difficult. However, the samples are shifted, so the averaging always sees all bursts and they are averaged once more in the higher layers. I think this has no influence on the final result.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035/1/src/osmo-bts-trx/scheduler_trx.c@1330">Patch Set #1, Line 1330:</a> <code style="font-family:monospace,monospace">LOGL_INFO</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Unrelated change.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035/1/src/osmo-bts-trx/scheduler_trx.c@1559">Patch Set #1, Line 1559:</a> <code style="font-family:monospace,monospace">RSSI_HR_MID_TWO</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Same question, why only two bursts? It should be 4 for a regular speech frame.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(see above)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035/1/src/osmo-bts-trx/scheduler_trx.c@1617">Patch Set #1, Line 1617:</a> <code style="font-family:monospace,monospace">LOGL_INFO</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Unrelated change.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/18035">change 18035</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bts/+/18035"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I902bb47d68742d2589156f61099b67a0edbaf40b </div>
<div style="display:none"> Gerrit-Change-Number: 18035 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 26 May 2020 21:08:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>