Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#8441 closed enhancement (needs_more_info)

aacenc: Add an option for max bitrate

Reported by: Lastique Owned by:
Priority: wish Component: avcodec
Version: git-master Keywords: aac
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Please, add an option for specifying the maximum allowed bitrate or the maximum encoded frame size for the built-in AAC encoder backend.

Limiting the maximum size of the encoded AAC frame is needed when the encoded frame needs to fit in a limited MTU.

Change History (7)

comment:1 by Carl Eugen Hoyos, 4 years ago

Keywords: aac added
Priority: normalwish
Version: unspecifiedgit-master

I believe this is missing a testcase that shows why such an option would be useful.

comment:2 by Carl Eugen Hoyos, 4 years ago

Resolution: needs_more_info
Status: newclosed

comment:3 by Lastique, 4 years ago

Resolution: needs_more_info
Status: closedreopened

I described the use case in the description - it would be useful to be able to limit AAC frames to fit in MTU for transmission over network or other media. For example, here is a PulseAudio Bluetooth A2DP backend for AAC codec, which needs this:

https://gitlab.freedesktop.org/andrey.semashev/pulseaudio/blob/a2dp-codecs-aac/src/modules/bluetooth/a2dp-codec-aac.c

The backend needs to limit AAC frames to MTU (minus RTP header size) because real hardware (Bluetooth headphones) don't support AAC frames larger than MTU.

comment:4 by Carl Eugen Hoyos, 4 years ago

Resolution: needs_more_info
Status: reopenedclosed

The testcase would consist of a simple command line together with the complete, uncut console output and an input sample.

comment:5 by Lastique, 4 years ago

Resolution: needs_more_info
Status: closedreopened

Are you seriously asking me to implement a console application that uses ffmpeg feature that is currently missing?

comment:6 by Carl Eugen Hoyos, 4 years ago

Resolution: needs_more_info
Status: reopenedclosed

comment:7 by Lastique, 4 years ago

You could just tell that you don't want to implement this feature, you know. Suit yourself, one user less.

Note: See TracTickets for help on using tickets.