Skip to content

xhe-aac: allow lower bit rates (mono 6 kbit/s, stereo 12 kbit/s) - #185

Merged
mstorsjo merged 2 commits into
mstorsjo:masterfrom
rafael2k:pr/xhe-low-bitrate
Nov 11, 2026
Merged

mstorsjo merged 2 commits into
mstorsjo:masterfrom
rafael2k:pr/xhe-low-bitrate

Conversation

@rafael2k

@rafael2k rafael2k commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

The open-source xHE-AAC encoder only carries the DASH configuration table, which starts at 12 kbit/s mono and 16 kbit/s stereo. This lowers the table minimums (two lines in main_aot_42_dash_oss.c).

Tested with the encoder and decoder round trip and with Dream's xHE-AAC receive path (tone test): mono 6, 8 and 10 kbit/s, stereo 12 kbit/s. Stereo 8 and 10 kbit/s cannot be opened by the encoder. Perceptual quality at the low rates has not been assessed.

Replaces #184, which was opened from a branch that also carried unrelated packaging commits.

🤖 Generated with Claude Code

The open-source build only has the DASH configuration table, which starts at
12 kbit/s mono and 16 kbit/s stereo. The same tools encode and decode lower
(tested with Dream's xHE-AAC receive path, tone test; perceptual quality at the
low rates has not been assessed): mono 6, 8 and 10 kbit/s, stereo 12 kbit/s.
Stereo 8 and 10 kbit/s cannot be opened by the encoder.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
At 48 kHz output (SBR 4:1, 12 kHz core) the mono SBR tuning table of the quad
rate mode starts at 6 kbit/s, so Open failed with an internal error (SBR
configuration) below that. Open the first row down to 2 kbit/s and the
configuration table row down to 4 kbit/s: mono 4 kbit/s now encodes and the
stream decodes (tested through a DRM chain at 4.5 kHz, tones and music; perceptual
quality at 4 kbit/s has not been assessed). 2 kbit/s is still rejected.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@mstorsjo mstorsjo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@mstorsjo
mstorsjo merged commit 5fae932 into mstorsjo:master Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants