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 744028 - video-converter: Converter doesn't work properly when offsets are specified
video-converter: Converter doesn't work properly when offsets are specified
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
unspecified
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-02-05 04:28 UTC by RaviKiran
Modified: 2015-02-06 10:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
do not use fastpath when offsets are specified (1.35 KB, patch)
2015-02-05 04:31 UTC, RaviKiran
none Details | Review

Description RaviKiran 2015-02-05 04:28:18 UTC
In videobox, I tried to use video-converter APIs for format conversions.
While creating a new converter, I provided configuration structure with 
GST_VIDEO_CONVERTER_OPT_SRC_X, SRC_Y, DEST_X and DEST_Y as offsets.
for pipeline, 
gst-launch-1.0 videotestsrc ! "video/x-raw,format={AYUV}" ! videobox left=-20 top=-20 ! "video/x-raw,format={BGRA}" ! videoconvert ! ximagesink
DEST_X = DEST_Y = 20

Expected result: Borders should be added at left and top.
Actual result: Borders were at right and bottom.

Video-converter fetches a fast path for this conversion, which uses ORC generated conversion function.
And they do not consider x and y offsets, hence borders weren't added at left and top.
Comment 1 RaviKiran 2015-02-05 04:31:40 UTC
Created attachment 296168 [details] [review]
do not use fastpath when offsets are specified

I do not know video-converter very well, but this patch worked for me.
Please review.
Comment 2 Wim Taymans 2015-02-05 14:07:05 UTC
Thanks, I changed it a little so that we can enable certain fastpaths again when we implement crop/border for them later.

commit 36a50778af85ad8b807e11f3f0bb9d67e1ac716a
Author: Wim Taymans <wtaymans@redhat.com>
Date:   Thu Feb 5 15:03:24 2015 +0100

    video-converter: disable fastpath for crop and border
    
    Add crop and border properties to the fastpath table and only select
    fastpath functions when it can handle the cropping or borders.
    
    Fixes https://bugzilla.gnome.org/show_bug.cgi?id=744028