After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 650914 - decklink sink has no audio support
decklink sink has no audio support
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal enhancement
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-05-23 21:41 UTC by Rafael Diniz
Modified: 2011-06-29 14:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
initial audio support for decklink cards (4.15 KB, patch)
2011-05-23 21:41 UTC, Rafael Diniz
none Details | Review
approach 1 (this one I get a fair sync when audio is present) no audio, no playback (11.47 KB, patch)
2011-05-26 12:52 UTC, Rafael Diniz
none Details | Review
approach 2 (this one does not really implements sync, but an other way to write the audio samples) (12.95 KB, patch)
2011-05-26 12:56 UTC, Rafael Diniz
none Details | Review

Description Rafael Diniz 2011-05-23 21:41:56 UTC
Created attachment 188423 [details] [review]
initial audio support for decklink cards

The audio pad of the decklink sink is not yet implemented.
I'd like to discuss here it's implementation as I write it.
Attached is an initial patch that includes audio support, but does not take care yet of synchronization.
Comment 1 Rafael Diniz 2011-05-26 12:47:35 UTC
I'll post here two approaches I'm writing.
Comment 2 Rafael Diniz 2011-05-26 12:52:00 UTC
Created attachment 188662 [details] [review]
approach 1 (this one I get a fair sync when audio is present) no audio, no playback
Comment 3 Rafael Diniz 2011-05-26 12:54:43 UTC
Comment on attachment 188662 [details] [review]
approach 1 (this one I get a fair sync when audio is present) no audio, no playback

This one make A/V in sync is video PTS is ahead of audio PTS inside the TS (which is always the case of a BTS, AFAIK). But do not cover all file cases.
Comment 4 Rafael Diniz 2011-05-26 12:56:21 UTC
Created attachment 188663 [details] [review]
approach 2 (this one does not really implements sync, but an other way to write the audio samples)
Comment 5 Rafael Diniz 2011-05-26 12:57:50 UTC
As I have 2 blackmagic cards in my machine and I wanted to use the second one, I setted:
int card_number = 1;

Please set this to 0 if you have only one card.
Comment 6 David Schleef 2011-06-25 18:32:06 UTC
commit bb4844b29eaf3551d32d95835ce77bf326687fbb
Author: Rafael Diniz <rafael@riseup.net>
Date:   Sun Jun 5 21:19:58 2011 -0700

    decklinksink: Add audio support
    
    Signed-off-by: David Schleef <ds@schleef.org>
Comment 7 Rafael Diniz 2011-06-29 14:36:28 UTC
The applied patch does not respect any PTS or timing at all.
Should I open a new bug with a name like:
"Decklink sink does not respect PTS"
?
Comment 8 Tim-Philipp Müller 2011-06-29 14:48:56 UTC
> Should I open a new bug with a name like:
> "Decklink sink does not respect PTS"

Yes, please!