[jitsi-dev] [jitsi/libjitsi] Auto keyframe control for recording (fast seek) (#184)


#1

This change enables video bridge users that solely use the bridge for
server side recording to enable auto keyframe control for faster
seeking in recorded files and automatically correct intermittent
issues. The function is controlled by a system parameter.
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/libjitsi/pull/184

-- Commit Summary --

  * Auto keyframe control for recording (fast seek)

-- File Changes --

    M src/org/jitsi/impl/neomedia/recording/WebmDataSink.java (27)

-- Patch Links --

https://github.com/jitsi/libjitsi/pull/184.patch
https://github.com/jitsi/libjitsi/pull/184.diff

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/libjitsi/pull/184


#2

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/libjitsi/pull/184#issuecomment-237564395


#3

Agreement already signed (btw the links are broken in the comment)

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/libjitsi/pull/184#issuecomment-237568129


#4

Yep, we know about the links, need update but it will happen next week.
Jenkins: add to whitelist

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/libjitsi/pull/184#issuecomment-237569284


#5

@zbettenbuk pushed 1 commit.

160ecc3 Fixing import format and comment

···


You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jitsi/libjitsi/pull/184/files/0e3907aa75f2a74991b1c1f20771f925231ed3a2..160ecc359827cf227270edfd5deb3d639373e784


#6

Merged #184.

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/libjitsi/pull/184#event-745472724