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 734711 - tsdemux: update output segments on seek
tsdemux: update output segments on seek
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-08-13 10:52 UTC by Vincent Penquerc'h
Modified: 2018-11-03 13:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
update output segments on seek (898 bytes, patch)
2014-08-13 10:52 UTC, Vincent Penquerc'h
needs-work Details | Review
update output segments on seek in pull mode (973 bytes, patch)
2014-08-15 14:53 UTC, Vincent Penquerc'h
none Details | Review
update output segments on seek in pull mode (1.05 KB, patch)
2014-08-15 15:36 UTC, Vincent Penquerc'h
none Details | Review

Description Vincent Penquerc'h 2014-08-13 10:52:15 UTC
Created attachment 283263 [details] [review]
update output segments on seek

This fixes at least the case where a non flushing rate changing seek
is done, but the new rate would not be communicated downstream.

I'm not sure whether it'd be best to only set this when the seek is non flushing, assuming that downstream will resend a segment when stopping flushing...
Comment 1 Sebastian Dröge (slomo) 2014-08-13 14:11:54 UTC
Comment on attachment 283263 [details] [review]
update output segments on seek

I think we should only do that in pull mode, otherwise upstream will send us a segment event after the seek which should then be transformed by tsdemux.
Comment 2 Edward Hervey 2014-08-14 16:18:35 UTC
This might be helpful ? It's a WIP patch to support updated TIME segments from upstream.

http://cgit.freedesktop.org/~bilboed/gst-plugins-bad/commit/?h=trickmodes&id=bcde22f243a4cc6e053b7d89a435ee021d237214
Comment 3 Vincent Penquerc'h 2014-08-15 14:53:45 UTC
Created attachment 283508 [details] [review]
update output segments on seek in pull mode

In pull mode only.

Shame that my original test case is in push mode though :p
Comment 4 Vincent Penquerc'h 2014-08-15 14:55:28 UTC
(In reply to comment #2)
> This might be helpful ? It's a WIP patch to support updated TIME segments from
> upstream.
> 
> http://cgit.freedesktop.org/~bilboed/gst-plugins-bad/commit/?h=trickmodes&id=bcde22f243a4cc6e053b7d89a435ee021d237214

Thanks - upstream time segment handling wouldnt help my particular case, as upstream here is merely filesrc + typefind.
Comment 5 Vincent Penquerc'h 2014-08-15 15:36:02 UTC
Created attachment 283515 [details] [review]
 update output segments on seek in pull mode

Updated, the stream pad can be NULL.
Comment 6 GStreamer system administrator 2018-11-03 13:26:03 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/169.