<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:</p><p style="white-space: pre-wrap; word-wrap: break-word;">My assumption was wrong, we already do send to BFIs in the current implementation:</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://gerrit.osmocom.org/c/osmo-bts/+/20981 osmo-bts-trx/scheduler: fix comments related to FACCH/H and BFI</p><p style="white-space: pre-wrap; word-wrap: break-word;">so why would we end up with one measurement sample less then? I guess the problem is that one such BFI is ignored by the measurement logic as expected, while another one is not, because it holds a valid RSSI value? During the last call, I proposed to ignore the measurement samples carried by BFIs triggered by FACCH, but what if we take the opposite approach and ignore FACCH samples instead? In other words, you count speech frames as usual, including all BFIs, but do not take FACCH samples into account. This way there would be need to mess up with expected number of measurements, it would always be constant. And of course, this is only in speech mode. In signalling, we count FACCH frames as usual. What do you think?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think your idea worked out. I have now BTS_Tests.TC_meas_res_speech_tchh_facch passing.<br>See also: https://gerrit.osmocom.org/c/osmo-bts/+/21024</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980">change 20980</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/+/20980"/><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: Ibf693aede8fffa6432cdcdcf5d52910493a1104b </div>
<div style="display:none"> Gerrit-Change-Number: 20980 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 02 Nov 2020 18:40:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>