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 666777 - [camerabin2] [PATCH] sync issues
[camerabin2] [PATCH] sync issues
Status: RESOLVED INCOMPLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks: 666718
 
 
Reported: 2011-12-23 20:57 UTC by Oleksij Rempel
Modified: 2016-02-21 23:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch, remove set_time and set_clock (964 bytes, patch)
2012-01-01 12:35 UTC, Oleksij Rempel
none Details | Review
patch v1 (4.64 KB, patch)
2012-03-26 14:37 UTC, Oleksij Rempel
none Details | Review

Description Oleksij Rempel 2011-12-23 20:57:03 UTC
i use custom version of cheese with camerabin and webm presets. This patch introduced some regression in my setup. After starting video record, thee video get sort of freezes... Produced file has audio without any delays, but with broken video.

The patch:
commit 0ce5021058e74bd956612c3b23f37582477a7a8d
Author: Thiago Santos <thiago.sousa.santos@collabora.com>
Date:   Mon Dec 19 16:49:08 2011 -0300

    camerabin2: Fix A/V sync for captures
    
    Audio src was pushing newsegments starting from 0, even when its buffers
    were not starting with 0 as their timestamps. Add a buffer probe that
    checks the ts of the first buffer and uses that to push a newsegment,
    just like we do for video on wrappercamerabinsrc.
Comment 1 Oleksij Rempel 2011-12-23 21:01:53 UTC
i use the camerabin2 patchset for cheese from this report:
https://bugzilla.gnome.org/show_bug.cgi?id=666718
Comment 2 Oleksij Rempel 2012-01-01 12:28:49 UTC
after some more playing around with camerabin2 i see it is not regression, this patch just showed existing bug.
For some reason the video and audio stream have different time stamps, after sencod "capture-start" video time stamps begin with 0. But Audio TS = running time of camerabin2.
The timestamps with first capture-start are not perfectly synced too, but is is not so bad.
Comment 3 Oleksij Rempel 2012-01-01 12:35:07 UTC
Created attachment 204412 [details] [review]
patch, remove set_time and set_clock

After removing gst_element_set_base_time and gst_element_set_clock for audio_src i got perfect synced V/A, but it is prefectly synced after second capture-start sequence. with first sequence there is still small delay.
Comment 4 Oleksij Rempel 2012-01-24 08:50:11 UTC
I use mkvinfo for testing resulted file.
Ways to reproduce:
- you should use "clean" muxer. muxer which do not change timestamps. for example matroskamux/webmmux
- use cheese (camerabin2 + vp8enc)
- capture some video
check it with:
mkvinfo -v file.webm
Comment 5 Oleksij Rempel 2012-03-26 14:37:43 UTC
Created attachment 210624 [details] [review]
patch v1

This is other way to fix sync issue. This patch filters new segment event. And send it from camerabin2.
Comment 6 Thiago Sousa Santos 2014-07-18 00:03:34 UTC
Is this still an issue with current git or a recent released version (1.3.90+)?
Comment 7 Tim-Philipp Müller 2016-02-21 23:27:35 UTC
Closing this bug report as no further information has been provided. Please feel free to reopen this bug report if you can provide the information that was asked for in a previous comment.
Thanks!