#11644: G722: encoder missing bitrates, broken decoding of all bitrates, not
gapless
-------------------------------------+-------------------------------------
Reporter: Balling | Owner: (none)
Type: defect | Status: new
Priority: normal | Component:
| undetermined
Version: unspecified | Resolution:
Keywords: | Blocked By:
Blocking: | Reproduced by developer: 0
Analyzed by developer: 0 |
-------------------------------------+-------------------------------------
Changes (by Balling):
* summary: G722: encoder missing bitrates, broke decoding, not gapless =>
G722: encoder missing bitrates, broken decoding of all bitrates, not
gapless
Old description:
> Summary of the bug:
>
> Ffmpeg code does not encode 48000 kbit or 56000 kbit and it does not
> decode those properly, even though it says it should (this is decoder
> documentation) https://www.ffmpeg.org/doxygen/0.11/g722_8c.html
>
> It is not gapless, at least on decoder side, and even more funny
> reference ITU code does not decode it properly.
> https://github.com/openitu/STL
> Oh and you need to force 16000 Hz before encoding G722 in ffmpeg, another
> bug. See e.g.
> How to reproduce:
> Compile https://github.com/openitu/STL/tree/dev/src/g722 and compare.
> Also see
> https://gitlab.freedesktop.org/pipewire/pipewire/-/merge_requests/2277 it
> dropped ffmpeg implementation.
>
> https://www.reddit.com/r/ffmpeg/s/rvuQEE3fjv Here is the info about
> 16000.
>
> Patches should be submitted to the ffmpeg-devel mailing list and not this
> bug tracker.
New description:
Summary of the bug:
Ffmpeg code does not encode 48000 kbit or 56000 kbit and it does not
decode those properly, even though it says it should (this is decoder
documentation) https://www.ffmpeg.org/doxygen/trunk/g722_8c.html
It is not gapless, at least on decoder side, and even more funny reference
ITU code does not decode it properly. https://github.com/openitu/STL
Oh and you need to force 16000 Hz before encoding G722 in ffmpeg, another
bug. See e.g. https://www.reddit.com/r/ffmpeg/s/rvuQEE3fjv
How to reproduce:
Compile https://github.com/openitu/STL/tree/dev/src/g722 and compare. Also
see https://gitlab.freedesktop.org/pipewire/pipewire/-/merge_requests/2277
it dropped ffmpeg implementation in favour of Android SpanDSP code.
Patches should be submitted to the ffmpeg-devel mailing list and not this
bug tracker.
--
--
Ticket URL: <https://trac.ffmpeg.org/ticket/11644#comment:2>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker
_______________________________________________
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac
To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".
