5G-MAG / rt-mbms-modem

MBMS - Modem
https://www.5g-mag.com/5gbroadcast
GNU Affero General Public License v3.0
15 stars 12 forks source link

Suspicious MCH BLER BER readings #38

Open DominiqueDePaepe opened 1 year ago

DominiqueDePaepe commented 1 year ago

Describe the bug BER Failed so far to find a single instance where a BLER > 0 would trigger BER also to turn different from zero. Such while we managed to create situations with totally different BLER levels from less than 1%, across a few percent, upto 10% and more. Hence I would appreciate to confirmation on this KPI to work and so how. if this MCH BER reading different from zero.

BLER Also it appears as if the the terminal is announcing way more frequently CRC errors, than we see BLER figure to show different from zero in the UI and the modem-measurements file.

To Reproduce Reduce incoming RF Power (and witness CINR drop ) till you see CRC errors come in. For myself when selected MCS23 such is at CINR of approx. 15 dB

Expected behavior

While understood that under excessive BLER, the device might not longer (be able to ) calculate the associated BER, such cannot be always the cases. Also we appreciate some clarification on how possible that BLER readings that are supposed to provide the average for the interval can show/stay zero while the terminal shows is announcing CRC errors already.

Screenshots (attached)

Desktop (please complete the following information): No dependency on desktop (same findings over the API)

Smartphone (please complete the following information): Not applicable

Additional context Add any other context about the problem here. image image image image

dsilhavy commented 1 year ago

@jsroldan Do you have an idea here?

jsroldan commented 1 year ago

Yes, the calculation of the BLER is not accurate in the rx right now. The rx miss some of the errors because how it stores and updates the variables. I already fix it, and when I finish I'll do a pull request.

dsilhavy commented 10 months ago

@jsroldan Do we have a PR for this?

dsilhavy commented 10 months ago

CAS Rel.16 + fixes will be included in the pull request that is going to be issued by @kuehnhammer .