[Asterisk-code-review] Adding tests that test forward error correction in codec opus (testsuite[master])

Kevin Harwell asteriskteam at digium.com
Thu Jan 19 13:11:15 CST 2017


Hello Mark Michelson, Anonymous Coward #1000019,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/4729

to look at the new patch set (#2).

Change subject: Adding tests that test forward error correction in codec_opus
......................................................................

Adding tests that test forward error correction in codec_opus

When FEC is enabled and negotiated in codec_opus dropped frames can be built
from a subsequently received frame. This series of tests establish a call
between two endpoints. Once the call is 'up' RTP packets are 'played' back
from a pcap file that contains roughly a minute of opus audio. Several packets
have been deleted in order to simulate dropped packets. The other end of the
call is monitored to make sure packets are received in the correct order and
with the appropriate timestamp/interval. If FEC is working properly there should
be no gaps in the sequence numbers of the received packets. Also the difference
between the timestamps should not vary.

These tests also make sure FEC works with and/or without jitter buffers enabled.

Change-Id: Ia01a0432cc23071836d594c7c3dc46709c9880bf
---
M lib/python/asterisk/pcap.py
A tests/codecs/opus/fec/16_bit_48khz_60_fec_dropped.pcapng
A tests/codecs/opus/fec/16_bit_48khz_60_fec_rtp.pcapng
A tests/codecs/opus/fec/jitterbuffer/adaptive/configs/ast1/codecs.conf
A tests/codecs/opus/fec/jitterbuffer/adaptive/configs/ast1/extensions.conf
A tests/codecs/opus/fec/jitterbuffer/adaptive/configs/ast1/pjsip.conf
A tests/codecs/opus/fec/jitterbuffer/adaptive/sipp/invite.xml
A tests/codecs/opus/fec/jitterbuffer/adaptive/sipp/invite_recv.xml
A tests/codecs/opus/fec/jitterbuffer/adaptive/test-config.yaml
A tests/codecs/opus/fec/jitterbuffer/fixed/configs/ast1/codecs.conf
A tests/codecs/opus/fec/jitterbuffer/fixed/configs/ast1/extensions.conf
A tests/codecs/opus/fec/jitterbuffer/fixed/configs/ast1/pjsip.conf
A tests/codecs/opus/fec/jitterbuffer/fixed/sipp/invite.xml
A tests/codecs/opus/fec/jitterbuffer/fixed/sipp/invite_recv.xml
A tests/codecs/opus/fec/jitterbuffer/fixed/test-config.yaml
A tests/codecs/opus/fec/jitterbuffer/tests.yaml
A tests/codecs/opus/fec/no_jitterbuffer/configs/ast1/codecs.conf
A tests/codecs/opus/fec/no_jitterbuffer/configs/ast1/extensions.conf
A tests/codecs/opus/fec/no_jitterbuffer/configs/ast1/pjsip.conf
A tests/codecs/opus/fec/no_jitterbuffer/sipp/invite.xml
A tests/codecs/opus/fec/no_jitterbuffer/sipp/invite_recv.xml
A tests/codecs/opus/fec/no_jitterbuffer/test-config.yaml
A tests/codecs/opus/fec/tests.yaml
M tests/codecs/opus/tests.yaml
A tests/codecs/rtp_analyzer.py
25 files changed, 890 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/29/4729/2
-- 
To view, visit https://gerrit.asterisk.org/4729
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia01a0432cc23071836d594c7c3dc46709c9880bf
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list