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 578656 - Implement upstream GstForceKeyUnit events in theoraenc
Implement upstream GstForceKeyUnit events in theoraenc
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-04-10 19:50 UTC by Olivier Crête
Modified: 2009-04-14 16:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
theoraenc: Factor out keyframe forcing (1.89 KB, patch)
2009-04-10 19:51 UTC, Olivier Crête
committed Details | Review
theoraenc: Implement upstream GstForceKeyUnit (3.82 KB, patch)
2009-04-10 19:51 UTC, Olivier Crête
committed Details | Review
draft-keyframe-force: Fix typo (1.00 KB, patch)
2009-04-10 19:51 UTC, Olivier Crête
committed Details | Review
draft-keyframe-force: Document upstream GstForceKeyUnit (1.26 KB, patch)
2009-04-10 19:51 UTC, Olivier Crête
committed Details | Review

Description Olivier Crête 2009-04-10 19:50:49 UTC
I'm attaching a series of patches that implement GstForceKeyUnit in theoraenc and update the documentation about it.

The way I implemented it, the upstream event travels up the pipeline until an element that takes as input data that has no keyframes (like raw data) swallows it and re-emits it as a synchronized downstream event downstream before the next frame.
Comment 1 Olivier Crête 2009-04-10 19:51:00 UTC
Created attachment 132489 [details] [review]
theoraenc: Factor out keyframe forcing
Comment 2 Olivier Crête 2009-04-10 19:51:02 UTC
Created attachment 132490 [details] [review]
theoraenc: Implement upstream GstForceKeyUnit
Comment 3 Olivier Crête 2009-04-10 19:51:04 UTC
Created attachment 132491 [details] [review]
draft-keyframe-force: Fix typo
Comment 4 Olivier Crête 2009-04-10 19:51:05 UTC
Created attachment 132492 [details] [review]
draft-keyframe-force: Document upstream GstForceKeyUnit
Comment 5 Wim Taymans 2009-04-12 20:39:53 UTC
Why does it need to send an event downstream as well? is there a downstream element that needs to do something special? Is it for when you have theoraenc ! oggmux and you need to also force a new 'keyframe' (page) out of the demuxer?
Comment 6 Olivier Crête 2009-04-12 21:08:17 UTC
I just send it downstream to match the behavior is the current downstream event..
Comment 7 Wim Taymans 2009-04-14 16:22:52 UTC
commit c628641880d9b0a1c7dacd2be8e67ecd47754d4e
Author: Olivier Crete <tester at tester.ca>
Date:   Tue Apr 14 18:16:37 2009 +0200

    theoraenc: implement upstream keyframe force
    
    Implement handling of upstream keyframe forcing.
    Update the design documents too.
    Fixes #578656

commit d0c36826278182b3bd986e6efafd65434127e26b
Author: Olivier Crete <tester at tester.ca>
Date:   Tue Apr 14 17:31:31 2009 +0200

    theoraenc: factor out keyframe forcing
    
    See #578656