[Bug 627459] theoraenc should provide option for TH_ENCCTL_SET_DUP_COUNT

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=627459
  GStreamer | gst-plugins-base | 0.10.x

--- Comment #19 from Sebastian Dröge <slomo at circular-chaos.org> 2011-05-09 08:37:27 UTC ---
(In reply to comment #16)
>
> So or so, the real frame is always on proper place. We create non existing
> frames from previous or current frame. So or so it can be not 100% sync with
> audio.

Well, you're creating a time machine here ;) Which is the main reason why it
feels wrong to me

(In reply to comment #17)
> IMHO, The latency is imaginer, because ogg do not use timestamps for video
> stream. We need only correct count of frames per second.

For creating files the one-frame latency is fine. Sure. But you also add this
latency to live video streams for example and, depending on the container
format, you'll also could violate bitrate constraints because you send the
duplicated frames (0 byte theora packets but N bytes container packets) in
bursts with the next non-dup frame.

(In reply to comment #18)
> Other two cents:
> - Frames miss the start of action. So it is only guess if action started right
> direct after 1 frame or direct before 2.

With your "time machine" it will actually be a bit harder... because from
looking at the video you don't know if you're looking at a
"backwards-replicated" dup frame or the real action.

> - stream compatibility. For computer forensics it will be a challenge. The
> stream created with Theora 1.1 and say Theora~1.5 (because the correct fix will
> be to change theora API) has different time recreation. Programs currently
> using old api even with new theora will use old api. So in my job (computer
> forensic ) i'll haw real problem to find what api was used to create current
> stream. And in what what the time recreation is wrong. So i think there is
> nothing to say against this argument.

Nothing to add to this comment, there's not only the "time machine" issue but
in forensic you'll also additionally have to consider the encoder version to
make accurate conclusions.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.