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 769193 - [REGRESSION] in NLE we are decoding both audio and video streams even if the we only care about one of those
[REGRESSION] in NLE we are decoding both audio and video streams even if the ...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-editing-services
unspecified
Other Linux
: Normal blocker
: 1.8.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-07-26 14:57 UTC by Thibault Saunier
Modified: 2016-07-26 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
uri-sources: Make sure to set decodebin 'caps' property (3.84 KB, patch)
2016-07-26 16:05 UTC, Thibault Saunier 		committed Details | Review

Description Thibault Saunier 2016-07-26 14:57:22 UTC 
This used to work (I am sure it was working when we worked on NLE so around 1.4), the regression must be either in decodebin or in NLE/GES.
Comment 1 Tim-Philipp Müller 2016-07-26 15:06:44 UTC 
Sure it wasn't 0.10? ;) I think decodebin has had the issue that it decodes unlinked streams for all of 1.x, unless you have special-handling with autoplug-continue (which ges doesn't seem to have).

Note that it might also depend on the container format, at least one demuxer has some nasty not-linked=deactivate stream code.
Comment 2 Tim-Philipp Müller 2016-07-26 15:07:37 UTC 
(Unless you mean higher-level, that nle sets up a decodebin for video even if you don't care about video in nle.)
Comment 3 Thibault Saunier 2016-07-26 16:05:08 UTC 
Created attachment 332145 [details] [review]
uri-sources: Make sure to set decodebin 'caps' property

Fixes a regression where we decode streams twice,
this was introduced when we started creating NLE
object at GESTrackElement construct time.

Fixes
Comment 4 Thibault Saunier 2016-07-26 16:08:17 UTC 
Attachment 332145 [details] pushed as 577938c - uri-sources: Make sure to set decodebin 'caps' property