Timothy B. Terriberry authored 1 year ago and Jean-Marc Valin committed 1 year ago

Compare the output of xcorr_kernel() against the results of
 xcorr_kernel_c() when configured with --enable-check-asm.
Currently this is only checked in fixed point, as a float check
 requires more sophisticated error analysis and may need to be
 customized for each vector implementation.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Still need more targeted tests, DRED decoding

Same for opus_multistream_packet_unpad()

Forgot to account for padding length bytes when DRED payload is large.

Fails ubsan because memcpy declares args as non-null

Matches the C version (see 4a7027b2)

luzpaz authored 1 year ago and Jean-Marc Valin committed 1 year ago

Found using `codespell -q 3 -L caf,highe,inlin,nd,ordert,shft`

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Rename, reindent, change arg order

Jean-Marc Valin authored 1 year ago and Jean-Marc Valin committed 1 year ago

Need to move the redundant frame even in CBR because the hybrid
frame now gets encoded as VBR, with DRED picking up the rest.
Fixes an issue introduced in 4600e775.

Jean-Marc Valin authored 1 year ago and Jean-Marc Valin committed 1 year ago

The encoder wouldn't reserve enough bits for CELT, causing it
to not have enough bits to code the switching redundancy flag
when it should have.

Adjust q0, qD and duration based on bitrate and loss.

No longer needed now that PLC is trained with PyTorch stack

Should handle the history in a more consistent way.
Slightly increase the model size and re-enable biased band loss in
training.

mostly untested

Should match the TF2 code, but mostly untested

Constrains the energy prediction to something safe.