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 575709 - Request a key frame
Request a key frame
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other All
: Normal minor
: 0.10.8
Assigned To: Wim Taymans
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-03-17 15:36 UTC by Janin Kolenc
Modified: 2010-05-10 10:45 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Adds support for this feature (2.93 KB, patch)
2009-03-17 15:47 UTC, Janin Kolenc
committed Details | Review
ffenc: Make upstream forcekeyunit event handling thread safe (1.81 KB, patch)
2009-04-14 19:21 UTC, Olivier Crête
none Details | Review

Description Janin Kolenc 2009-03-17 15:36:24 UTC
Please describe the problem:
There are some cases when you need to request a key frame from the encoder (Video Fast Update, for example) this is not currently possible with gst-ffmpeg. The only encoder which supports it is theora encoder in gst-plugins-base. It would be nice to have this in ffmpeg as well.

Steps to reproduce:
 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Janin Kolenc 2009-03-17 15:47:17 UTC
Created attachment 130833 [details] [review]
Adds support for this feature
Comment 2 Edward Hervey 2009-03-17 15:54:29 UTC
Looks good to me. Unless wtay has some other input on this it can be committed after the freeze.
Comment 3 Wim Taymans 2009-03-17 18:57:28 UTC
looks good to me.
Comment 4 Wim Taymans 2009-03-25 11:53:51 UTC
commit a44f5b4df3c7c806820358d0c4913cc53c44da46
Author: Janin Kolenc <janin dot kolenc at marand dot si>
Date:   Wed Mar 25 12:51:55 2009 +0100

    ffenc: Add support for I-frame request.
    
    Add request for I-frame request using GstForceKeyUnit event.
    Fixes #575709.
Comment 5 Olivier Crête 2009-04-14 19:21:23 UTC
Created attachment 132654 [details] [review]
ffenc: Make upstream forcekeyunit event handling thread safe

The current code can modify the struct passed to ffmpeg in the srcpad event handler, since its not under the stream lock. Attach patch should fix that.
Comment 6 Mark Nauwelaerts 2010-05-10 10:45:31 UTC
Above patch is included in:

commit b69fcc471ea268587a8796ddffc888f321933d0a
Author: Olivier Crête <olivier.crete@collabora.co.uk>
Date:   Fri Nov 20 21:46:33 2009 -0500

    ffenc: Make keyframe forcing thread-safe

    Make the keyframe forcing thread-safe, also emit a downstream event
    to allow muxers or payloaders to react appropriately.

    https://bugzilla.gnome.org/show_bug.cgi?id=602556