Koen Vos authored 13 years ago and Jean-Marc Valin committed 13 years ago

Simplifies mono/stereo switching in SILK
Fixes a quantization mismatch between encoder and decoder
Constrains the pitch lags in the same way in the encoder and decoder

Koen Vos authored 13 years ago and Jean-Marc Valin committed 13 years ago

Fixes a mono-stereo switching glitch and fixes a side prediction
bug where coding side would result in worse panned-mono behaviour

Gregory Maxwell authored 13 years ago and Jean-Marc Valin committed 13 years ago

There were assert failures for test cases with mode switching and LBRR enabled.
(some similar problems remain)

Tim identified that the assert was falsely firing while encoding LBRR frames but
the flag doesn't matter while encoding the bitstream it only actually matters in
silk/process_NLSFs.c.

custom modes.

More assert updates, make the initialization in silk_warped_autocorrelation_FLP more explicit to make tools happy, and allow resetting to auto with OPUS_SET_FORCE_CHANNELS.

We now reset the CELT state when swithing to hybrid and we "drain"
the CELT MDCT overlap when switching away from hybrid.

Gregory Maxwell authored 13 years ago and Jean-Marc Valin committed 13 years ago

Add the noreturn attribute on the assert functions to aid static analysis, improve test_repacketizer error handling, and silence 19 clang static analysis errors with additional assertions.

Gets rid of some more glitches caused by the decoder resampler

The redundancy_bytes<0 test always fails because of the ec_tell(&dec)+29 test above.

Conflicts:
	src/opus_encoder.c

Delaying stereo->mono switching decisions so that SILK can do a smooth
downmix. Also, wrote proper float/fixed code for the hybrid variable
stereo collapse, including a smooth downmix for stereo<->mono switching

Fix bug in the OPUS_GET_APPLICATION_REQUEST ctl and also add a bunch of set ctl range checking and improve encoder/decoder error code consistency.

Gregory Maxwell authored 13 years ago and Jean-Marc Valin committed 13 years ago

As of commit 44203907 all celt bands N>1 are even, including custom modes. In light of this, remove some pointless tests.

Jean-Marc Valin authored 13 years ago and Jean-Marc Valin committed 13 years ago

LPC_inverse_pred_gain_QA() was failing to detect unstable filters
because of insufficient numerical precision. Precision was increased
to Q24 for LPC coefs and the reciprocal now uses all the precision
available (variable shift right from the start).

(Fs_kHz is always 8/12/16 in Opus and also enforced by the assert on line 112)

There's still a range coder mismatch on the first frame when using FEC.

Timothy B. Terriberry authored 13 years ago and Jean-Marc Valin committed 13 years ago

The SILK bitstream allowed coding 0 pulses in a shell block, but a
 non-zero number of LSb's, meaning some excitation coefficients
 could be non-zero, but would not have a corresponding sign.
To fix this without breaking already-encoded bitstreams, this patch
 adds a set of sign PDFs for the 0 pulses case.

This is occasionally more efficient than the normal encoding if
 there are a large number of coefficients with positive signs,
 since these cost more than 1 bit when using the > 0 pulse PDFs.
It only saves 0.33 bits per second (on average: it does better at
 high rates), but that's probably enough to justify the two
 redundant ways of coding things (and it's too late now to remove
 the second one entirely, anyway).
This patch does not include the encoder modifications required to
 check if this coding method is more efficient and switch to it.