Attention is currently required from: laforge, pespin, dexter.
fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/27555 )
Change subject: osmo-bts-trx: rx_{tchf,tchh}_fn(): ensure complete set of bursts
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> I'm wondering if there aren't speech modes where codec blocks with missing
> bursts still contain sufficient data (after decode) to produce something
> useful?
Correct, there are. But I think it's not that critical if we loose *just one*
speech frame after the channel activation. What we benefit from this approach
is the ability to quickly lookup TDMA frame number from the measurement history
(see Id9a2b7b0f1a1ad7cfbbab862faf521e135c90605), and also eliminate one or two
measurement samples in the beginning with BER10k>0 for TCH/F and TCH/H,
respectively.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/27555
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I56bebe1374eb803e3c1e9f08dda4da50a074ab0b
Gerrit-Change-Number: 27555
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Fri, 01 Apr 2022 02:20:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment
