Project

General

Profile

Actions

Bug #3510

closed

Make sure the ECU (Error Concealment Unit) is working correctly

Added by fixeria over 5 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
osmo-bts-trx
Target version:
-
Start date:
08/29/2018
Due date:
% Done:

0%

Spec Reference:

Description

In 69d0d506775c82eb2bde66fe748100a94a3173a0 "osmo-bts-trx: perform error concealment for FR frames",
the ECU (Error Concealment Unit) was introduced. In short, if one (or more) speech frame is lost,
one may experience some unpleasant audio effects. The ECU is used to avoid such effects.

While working on audio support in OsmocomBB, I have discovered that the libosmocoding API
actually produces decoded speech frames in RTP format, and I guess the ECU implementation
may expect speech frames in canonical format.

I think it makes sense to:

  • clarify, which frame format is expected by the libosmocodec's ECU FR implementation?
    • add some comments there (I couldn't find any);
  • manually test with a regular phone (by dripping TCH bursts somehow);
  • add some TTCN-3 testing coverage;
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)