Actions
Bug #3510
closedMake sure the ECU (Error Concealment Unit) is working correctly
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