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 303727 - [PATCH] add a ffscale element
[PATCH] add a ffscale element
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal enhancement
: 0.8.5
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-05-10 21:22 UTC by Luca Ognibene
Modified: 2005-06-30 16:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to add ffscale to gst-ffmpeg (13.43 KB, patch)
2005-05-10 21:23 UTC, Luca Ognibene
none Details | Review

Description Luca Ognibene 2005-05-10 21:22:50 UTC
This patch adds a ffscale element to gst-ffmpeg. The element is based on
imgresample and can resize video. Right now there are other 2 similar elements
in gstreamer: videoscale and gdkpixbufscale, some things about these elements:

* ffscale is faster than gdkpixbufscale (a lot faster)
* ffscale is slower than videoscale* ffscale images are the same quality as
gdkpixbuscale's one
* videoscale has really bad quality (see #161079)
* ffscale supports only I420
* gdkpixbuscale supports only RGB

I'm attaching here for review.. and to see if someone other than me would like
to see this element in cvs :)
Comment 1 Luca Ognibene 2005-05-10 21:23:28 UTC
Created attachment 46307 [details] [review]
patch to add ffscale to gst-ffmpeg
Comment 2 Ronald Bultje 2005-05-13 11:04:40 UTC
Well, it's in the TODO in gst-ffmpeg, so yes, I've considered this cool-to-have
at some point, too. :). Not sure how useful it is in the end for mainstream
(like Totem) because ffmpeg limits itself to YUV420, but well, why not...
Deinterlacing would rock, too (see TODO once more).

Apply it, I'd say, although long-term it's probably better to work on
videoscale... Up to you, though.

Thanks.
Comment 3 Luca Ognibene 2005-05-13 15:49:36 UTC
committed as ffvideoscale (as written in the TODO)