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 703712 - avvideoscale negotiation error
avvideoscale negotiation error
Status: RESOLVED DUPLICATE of bug 742817
Product: GStreamer
Classification: Platform
Component: gst-libav
1.1.1
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-06 14:49 UTC by RajuB
Modified: 2015-02-24 12:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
In the attached file I have fixed the issue described above (21.44 KB, patch)
2013-07-06 14:49 UTC, RajuB
none Details | Review
the patch contains the solution for the bug reported (704 bytes, patch)
2013-07-06 15:16 UTC, RajuB
none Details | Review
sorry for all the mess. please consider the latest patch (683 bytes, patch)
2013-07-06 15:23 UTC, RajuB
none Details | Review

Description RajuB 2013-07-06 14:49:39 UTC
Created attachment 248524 [details] [review]
In  the attached file I have fixed the issue described above

I am using gstreamer-1.1.1 version. When I was testing the pipeline with following command, I found negotiation errors.

gst-launch-1.0 filesrc location=720p_504f.yuv ! videoparse width=1280 height=720  ! avvideoscale ! video/x-raw,width=176,height=144  ! filesink location=176x144_504f.yuv

To reproduce this issue, run above command with any 720P yuv input.

Later I further debugged this issue and found the solution. I have attached a patch for this issue. Can anybody confirm the issue and patch??
Comment 1 RajuB 2013-07-06 15:16:12 UTC
Created attachment 248525 [details] [review]
the patch contains the solution for the bug reported

Discard my previous attachment.
Comment 2 RajuB 2013-07-06 15:20:22 UTC
Comment on attachment 248525 [details] [review]
the patch contains the solution for the bug reported

307a308,309
>   /* Fix for negotiation error */
>   othercaps = gst_caps_truncate (othercaps);
446c448
< gst_ffmpeg_caps_to_pixfmt (const GstCaps * caps)
---
> gst_ffmpeg_caps_to_pixfmt (GstVideoInfo info)
448d449
<   GstVideoInfo info;
451,455d451
<   GST_DEBUG ("converting caps %" GST_PTR_FORMAT, caps);
< 
<   if (gst_video_info_from_caps (&info, caps))
<     goto invalid_caps;
< 
537,538c533,534
<   scale->in_pixfmt = gst_ffmpeg_caps_to_pixfmt (incaps);
<   scale->out_pixfmt = gst_ffmpeg_caps_to_pixfmt (outcaps);
---
>   scale->in_pixfmt = gst_ffmpeg_caps_to_pixfmt (scale->in_info);
>   scale->out_pixfmt = gst_ffmpeg_caps_to_pixfmt (scale->out_info);
Comment 3 RajuB 2013-07-06 15:23:25 UTC
Created attachment 248526 [details] [review]
sorry for all the mess. please consider the latest patch
Comment 4 Sebastian Dröge (slomo) 2013-07-09 09:01:15 UTC
Please attach a patch in "git format-patch" style or at least a unified diff.
Comment 5 RajuB 2013-07-09 14:29:38 UTC
(In reply to comment #4)
> Please attach a patch in "git format-patch" style or at least a unified diff.

Could you please tell me how to generate "git format-patch" style??
Comment 6 Tim-Philipp Müller 2013-07-09 14:49:15 UTC
You commit it locally in your git checkout. It should be the first entry in "git log" then. Make sure your name + e-mail address is set properly, and you have a commit message.

Then you do

  git format-patch -1

which should create a file called 0001-*.patch which you then attach to this bug.
Comment 7 Tim-Philipp Müller 2015-02-24 12:46:25 UTC
Thanks for taking the time to report this.
This particular bug has already been reported into our bug tracking system, but please feel free to report any further bugs you find.

*** This bug has been marked as a duplicate of bug 742817 ***