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 785119 - mxfdemux: Wrong PTS on some files
mxfdemux: Wrong PTS on some files
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal normal
: 1.12.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-07-19 11:34 UTC by Edward Hervey
Modified: 2017-07-25 08:51 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Edward Hervey 2017-07-19 11:34:29 UTC
The following file (in validate media test suite) ends up outputting wrong PTS.

gst-launch-1.0 -v file:///$HOME/gst-validate/gst-integration-testsuites/medias/defaults/mxf/op2b-mpeg2-wave_hd.mxf ! mxfdemux ! video/mpeg ! fakesink silent=False

It starts from
* dts: 0:00:00.000000000, pts: 0:00:00.040000000
all the way to
* dts: 0:00:01.880000000, pts: 0:00:01.800000000
But then it carries on with
* dts: 0:00:01.920000000, pts: 0:00:00.040000000
* dts: 0:00:01.960000000, pts: 0:00:00.080000000
....
Comment 1 Sebastian Dröge (slomo) 2017-07-19 12:58:02 UTC
Makes sense. We need to keep the offset when switching between essence tracks
Comment 2 Sebastian Dröge (slomo) 2017-07-24 13:50:14 UTC
commit c2ef63a3bc93358250b203fc5a40ade7416e7a6e
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Mon Jul 24 16:21:31 2017 +0300

    mxfdemux: Keep track of component start positions and material track positions
    
    This allows us to know exactly where in the material track we are, and
    how to convert from a PTS for a source track to the actual PTS of the
    material track (i.e. by adding the component start position).
    
    https://bugzilla.gnome.org/show_bug.cgi?id=785119
Comment 3 Sebastian Dröge (slomo) 2017-07-24 14:00:32 UTC
commit 59f9bc05d29038e498ed4c96311adc567fe57bfd
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Mon Jul 24 16:54:22 2017 +0300

    mxfdemux: Use material track edit rate for component PTS offsetting