h.264 encoding does not work with newest FFmpeg

Bug #518578 reported by ptf
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenShot Video Editor
Fix Released
Critical
Jonathan Thomas

Bug Description

Openshot 1.0, 1.0.5; Fedora 12; mlt 4.10

Selecting h.264 as the video codec (with any offered container type) produces a file with sound, but no video.

There is also no way to set any of the encoder parameters (this is a general problem BTW) and ffmpeg is known to set bogus defaults.

When this works support for a lossless h.264 profile would be very useful.

Tags: patch
Revision history for this message
ptf (paul-flinders) wrote :

OK - I've confirmed the problem seems to be that the codec parameters aren't set, attached is a quick-and-dirty patch which "fixes" libx264 output by setting up the codec parameters.

After applying the patch choosing x.264 video within an mp4 container (haven't tried other containers yet) produces a file with both audio *and* video (which plays OK in mplayer).

Although you could, perhaps, use this for v1.1 I think that there should really be some way of editing these. Certainly for x.264 and probably there should be a more generic "advanced codec configuration" dialog.

Revision history for this message
Olivier Girard (eolinwen) wrote :

Thanks a lot for the patch. Do you test the others containers ? It will a great helpfull.
Thanks

Revision history for this message
hva (francesco-hermanitosverdes) wrote :

Your patch seems to work for me as well, output with libx264 is fixed

Revision history for this message
Olivier Girard (eolinwen) wrote :

I have forgotten to tell that's will be included in the next version.

Revision history for this message
ptf (paul-flinders) wrote :

> I have forgotten to tell that's will be included in the next version.

Ok, then the same change probably needs to be made in cli_render.py as well.

Revision history for this message
hva (francesco-hermanitosverdes) wrote :

correct, herre it is a patch that's supposed to fix h264 output in command-line rendering too.

Revision history for this message
Jonathan Thomas (jonoomph) wrote :

I need to test this patch against the current versions of ffmpeg and mlt, to make sure it won't break all of the users on older versions. If all tests work, then we can merge this into the 1.1 code. Thanks!

Changed in openshot:
milestone: none → 1.2.0
milestone: 1.2.0 → 1.1.0
Revision history for this message
ptf (paul-flinders) wrote :

I'm running mlt 4.10 and ffmpeg-0.5-6.20100116svn from rpmfusion - that's about as current as you can get without pulling development versions (I've done that too but not run openshot against the result yet).

Revision history for this message
GMaq (info-bandshed) wrote :

Hello,

Have these patches been integrated into 1.1 Alpha1? On AV Linux 3.0R1 (Debian Testing) all x264 exports from Openshot are still not working (audio with no video) using ffmpeg 0.5+svn20100126, libx264-83, and melt 0.4.10 on Openshot 1.1 Alpha1. I would suggest this is a fairly urgent fix as x264 in all it's various forms is a very common export format.

Revision history for this message
Jonathan Thomas (jonoomph) wrote :

No, the patch has not been integrated into 1.1 yet. But it is planned very soon. =)

Revision history for this message
GMaq (info-bandshed) wrote :

Jonathan,

That's great news! I can't wait.

@ptf Thanks so much for providing these patches, IMHO this has been a big one to get sorted out.

In the meantime forgive my ignorance but how can these patches be integrated into an existing Openshot install?

Revision history for this message
Olivier Girard (eolinwen) wrote :

Hello GMag
Find the last version of openshot by :
bzr branch lp:openshot
go in the folder
cd openshot
download le patch
wget http://launchpadlibrarian.net/39067639/render_options_for_x264.patch (or by another way)
add the patch by :
patch -p 0 < render_options_for_x264.patch
after
python setup.py build
cd bin
./openshot
I hope to be clear, i'm not a specialist and you have more explanations here : http://openshotusers.com/forum/viewtopic.php?f=11&t=204

Revision history for this message
Olivier Girard (eolinwen) wrote :

I have just seen that Jonathan had merged the patch in the branch. So, you have just to get back the last version on launchpad.
Thanks and Great distribution (AVLinux). Could you create a torrent please ? Thanks.

Revision history for this message
GMaq (info-bandshed) wrote :

cenwen

Thanks for all your help! I'll look into a torrent in the near future.

Thanks Jonathan, an upstream donation from AV Linux is coming your way soon!

Revision history for this message
Jonathan Thomas (jonoomph) wrote :

Seems to work great. I have committed this to the trunk. Thanks!

Changed in openshot:
assignee: nobody → Jonathan Thomas (jonoomph)
status: New → Fix Committed
importance: Undecided → Critical
summary: - h.264 output still broken
+ h.264 encoding does not work with newest FFmpeg
moimael (moimael)
Changed in openshot:
status: Fix Committed → Fix Released
tags: added: patch
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.