This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.
It is not clear why appending a coded frame whose decode timestamp is between the last decode timestamp and last decode timestamp + 100ms should trigger an error. What is the rationale for '100 ms' ? Also, since the tests are made in decode order, maybe the Append Sequence definition should be clarified to say "monotonically increasing in *decode* time".
(In reply to comment #0) > It is not clear why appending a coded frame whose decode timestamp is > between the last decode timestamp and last decode timestamp + 100ms should > trigger an error. What is the rationale for '100 ms' ? Like the note states, this is to detect out of order appends. 100ms seemed like a reasonable default that allows a little bit of flexibility on how far apart coded frames are. > > Also, since the tests are made in decode order, maybe the Append Sequence > definition should be clarified to say "monotonically increasing in *decode* > time". I'll add this in the next spec update.
(In reply to comment #1) > (In reply to comment #0) > > It is not clear why appending a coded frame whose decode timestamp is > > between the last decode timestamp and last decode timestamp + 100ms should > > trigger an error. What is the rationale for '100 ms' ? > > Like the note states, this is to detect out of order appends. 100ms seemed > like a reasonable default that allows a little bit of flexibility on how far > apart coded frames are. I thought the note was about the first part (decode timestamp < last decode timestamp). It's a bit akward to me to have to rely on hard-coded numbers to detect out-of-order appends. What if you have very low frame rates? Why not have a gap-threshold attribute or use something like last decode timestamp + 2xduration of the previous frame? > > > > Also, since the tests are made in decode order, maybe the Append Sequence > > definition should be clarified to say "monotonically increasing in *decode* > > time". > > I'll add this in the next spec update. Thanks.
Changes committed. https://dvcs.w3.org/hg/html-media/rev/1e6898152c5b Decode time clarification to Append Sequence added.
Change committed. https://dvcs.w3.org/hg/html-media/rev/f7f2b7226543 Converted 100ms limit to 2x last frame duration limit.