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 751647 - audio/video: Copy over more metas by default
audio/video: Copy over more metas by default
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 742385
Blocks:
 
 
Reported: 2015-06-29 13:24 UTC by Sebastian Dröge (slomo)
Modified: 2018-11-03 11:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
adapter: Copy over GstMeta from the input buffers to the output (3.08 KB, patch)
2015-06-29 15:04 UTC, Sebastian Dröge (slomo)
rejected Details | Review

Description Sebastian Dröge (slomo) 2015-06-29 13:24:57 UTC
For audio/video decoder/encoder/filter/etc we're currently copying all metas without tags and with audio/video tags only.

We could also copy over things with e.g. the video-dimensions tag if the input and output caps are the same, or even in other cases by calling the meta's transform function.
Comment 1 Sebastian Dröge (slomo) 2015-06-29 15:04:54 UTC
Created attachment 306299 [details] [review]
adapter: Copy over GstMeta from the input buffers to the output

All functions that return a GstBuffer or a list of them will now copy
all GstMeta from the input buffers except for meta with GST_META_FLAG_POOLED
flag or "memory" tag.

This is similar to the existing behaviour that the caller can't assume
anything about the buffer flags, timestamps or other metadata.
Comment 2 Sebastian Dröge (slomo) 2015-06-29 15:11:45 UTC
Comment on attachment 306299 [details] [review]
adapter: Copy over GstMeta from the input buffers to the output

Wrong bug :)
Comment 3 Sebastian Dröge (slomo) 2015-07-01 10:13:06 UTC
As discussed on IRC, elements should do the following:

- If all tags of a meta are known, and known to not be affected by whatever the element does, then the COPY transform should be called
- Otherwise one or more non-COPY transform that correspond to whatever the element does should be called on the meta


We will need to add more meta transforms, and probably also more tags.


Also videocrop/roi are currently having wrong tags or incomplete tags at least: video, video-size. However the size can stay the same but still the meta can become invalid, e.g. when keeping the size but cropping+scaling the content.
Comment 4 GStreamer system administrator 2018-11-03 11:39:08 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-base/issues/204.