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 668564 - avimux: Add support for raw RGB video
avimux: Add support for raw RGB video
Status: RESOLVED DUPLICATE of bug 695543
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Windows
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-01-24 11:13 UTC by Petteri Aimonen
Modified: 2016-11-06 20:46 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (1.28 KB, application/octet-stream)
2012-01-24 11:13 UTC, Petteri Aimonen 		  Details
patch, version 2 (4.48 KB, patch)
2012-01-27 09:37 UTC, Petteri Aimonen 		none Details | Review

Description Petteri Aimonen 2012-01-24 11:13:54 UTC 
Created attachment 205962 [details]
patch

The attached patch adds support for uncompressed RGB24 format in gstavimux.
Comment 1 Petteri Aimonen 2012-01-27 09:37:59 UTC 
Created attachment 206247 [details] [review]
patch, version 2

Apparently I had managed to stumble upon a format (RV24) that was supported only by VLC. Making proper DIB images turned out to be a bit more complex, but the new patch does that.

This time the output plays back properly in VLC, Windows Media Player and VirtualDub.
Comment 2 Julien Isorce 2012-01-27 10:34:17 UTC 
Hi, I am wondering if avi spec supports negative height.

If yes then you should try to set a negative height in the header instead of inverting DIB . But it supposed that VLC, WMP and VirtualDub supports it too (it should if this is in the avi spec). (And handle this in avidemux too).
Comment 3 Petteri Aimonen 2012-02-03 09:26:48 UTC 
Hmm, using negative height didn't seem to affect the playback in VLC, WMP nor VirtualDub. I.e. it did not flip the video, which would have been desirable. It does cause gstreamer to spit out an error when trying to play the file, though.

There are however many fields in the AVI header which contain the height - I tried the most obvious combinations negative/positive values, but not all. I don't know the format well enough to comment on whether the negative height should be allowed and how it should behave.

I think that when producing AVI files, it is best to use positive height and reverse the scanlines manually (like the patch does). This ensures the best compatibility across programs.
Comment 4 Tim-Philipp Müller 2016-11-06 20:46:13 UTC 
Thanks for the patch, and sorry no one ever got to merging it, but it looks like this has since been fixed independently.

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