opusdec: add 'stats' property (!641) · Merge requests · GStreamer / gst-plugins-base · GitLab

Snippets Groups Projects

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

Merged Guillaume Desmottes requested to merge gdesmott/gst-plugins-base:opusdec-stats into master 4 years ago

Allow users to retrieve the number of samples, and their duration, generated using PLC.

Edited 4 years ago by Olivier Crête

Activity

Guillaume Desmottes @gdesmott · 4 years ago

Author Developer

See also gst-plugins-bad!1210 (merged)

I also wrote a simple tool for testing: https://gitlab.freedesktop.org/gdesmott/gst-test-plc
Guillaume Desmottes mentioned in merge request gst-plugins-bad!1210 (merged) 4 years ago

mentioned in merge request gst-plugins-bad!1210 (merged)
Stéphane Cerveau @dabrain34 · 4 years ago

Reporter

it would be nice to have a unit test
Olivier Crête changed title from opusdoc: add 'stats' property to opusdec: add 'stats' property 4 years ago

changed title from opusdoc: add 'stats' property to opusdec: add 'stats' property
Olivier Crête changed the description 4 years ago

changed the description
Olivier Crête @ocrete · 4 years ago

Maintainer

Resolved 4 years ago by Guillaume Desmottes

Can you also add the number of packets, etc? Maybe some info from the caps? Like which sample rate is being produced, is it mono or stereo? I'm not sure if there is any other useful stats? Maybe also some stats from libopus, I see the pitch and the bandwidth from OPUS_GET_BANDWIDTH/OPUS_GET_PITCH from opus_decoder_ctl() ?

Last reply by Guillaume Desmottes 4 years ago
Nicolas Dufresne @ndufresne · 4 years ago

Maintainer

Resolved 4 years ago by Guillaume Desmottes

Please update your commit message too, opusdoc->opusdec
Guillaume Desmottes added 1 commit 4 years ago
added 1 commit

6c1b8258 - opusdec: add 'stats' property

Compare with previous version
Guillaume Desmottes added 1 commit 4 years ago
added 1 commit

01e4d8d8 - opusdec: add 'stats' property

Compare with previous version
Guillaume Desmottes resolved all threads 4 years ago

resolved all threads
Tim-Philipp Müller @tpm started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by Guillaume Desmottes
Last reply by Guillaume Desmottes 4 years ago

Guillaume Desmottes added 3 commits 4 years ago

added 3 commits

01e4d8d8...a4e49ba8 - 2 commits from branch gstreamer:master
404be0cb - opusdec: add 'stats' property

Compare with previous version

Guillaume Desmottes resolved all threads 4 years ago

resolved all threads

Guillaume Desmottes assigned to @gstreamer-merge-bot 4 years ago

assigned to @gstreamer-merge-bot

Guillaume Desmottes mentioned in commit 4 years ago

mentioned in commit gdesmott/gst-plugins-base@de2be15a

GStreamer Marge Bot added 1 commit 4 years ago

added 1 commit

de2be15a - opusdec: add 'stats' property

Compare with previous version

GStreamer Marge Bot assigned to @gdesmott and unassigned @gstreamer-merge-bot 4 years ago

assigned to @gdesmott and unassigned @gstreamer-merge-bot

GStreamer Marge Bot @gstreamer-merge-bot · 4 years ago

Developer

I couldn't merge this branch: CI failed!

Guillaume Desmottes added 1 commit 4 years ago

added 1 commit

2fcdb8a8 - opusdec: add 'stats' property

Compare with previous version

Guillaume Desmottes assigned to @gstreamer-merge-bot and unassigned @gdesmott 4 years ago

assigned to @gstreamer-merge-bot and unassigned @gdesmott

Please register or sign in to reply