This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.
Below are miscellaneous comments and/or questions. > The fade in coded frame equals new coded frame. What about if new coded frame duration is less than 5 ms? > Convert fade out samples and fade in samples to a common > sample rate and channel layout. The specification should mandate no sample rate conversion unless necessary. >Apply a linear gain fade out From where to where? > the difference between decode timestamp and last decode timestamp is greater than 100 milliseconds Why 100 ms?
Retitling bug to describe comment.
(In reply to comment #0) > Below are miscellaneous comments and/or questions. > > > The fade in coded frame equals new coded frame. > > What about if new coded frame duration is less than 5 ms? This is covered in the audio splice rendering algorithm, but I will also add a note in step 13 of the Audio Splice Frame Algorithm calling out this case. I don't want to handle this explicitly in the algorithm because it will just make things unnecessarily confusing. > > > Convert fade out samples and fade in samples to a common > > sample rate and channel layout. > > The specification should mandate no sample rate conversion unless necessary. Does this really need to be explictly called out? > > >Apply a linear gain fade out > > From where to where? What do you mean? If this isn't clear enough please provide text that you consider more clear. I assumed that this plus the picture in the spec was sufficient to convey the intended meaning. > > > the difference between decode timestamp and last decode timestamp is greater than 100 milliseconds > > Why 100 ms? It seemed like a reasonable default. It is an arbitrary value chosen to ensure that coded frames can't be too far apart before an out-of-order append error is signalled.
Changes committed. https://dvcs.w3.org/hg/html-media/rev/1e6898152c5b Clarified what to do if 'new coded frame' is less than 5ms long.
> > The specification should mandate no sample rate conversion unless necessary. > > Does this really need to be explictly called out? Yes. > > >Apply a linear gain fade out > > > > From where to where? > > What do you mean? I think the specification should at least recommend a fade in/out slope. (In reply to comment #2) > (In reply to comment #0) > > Below are miscellaneous comments and/or questions. > > > > > The fade in coded frame equals new coded frame. > > > > What about if new coded frame duration is less than 5 ms? > > This is covered in the audio splice rendering algorithm, but I will also add > a note in step 13 of the Audio Splice Frame Algorithm calling out this case. > I don't want to handle this explicitly in the algorithm because it will just > make things unnecessarily confusing. > > > > > > Convert fade out samples and fade in samples to a common > > > sample rate and channel layout. > > > > The specification should mandate no sample rate conversion unless necessary. > > Does this really need to be explictly called out? > > > > > >Apply a linear gain fade out > > > > From where to where? > > What do you mean? If this isn't clear enough please provide text that you > consider more clear. I assumed that this plus the picture in the spec was > sufficient to convey the intended meaning. > > > > > > the difference between decode timestamp and last decode timestamp is greater than 100 milliseconds > > > > Why 100 ms? > > It seemed like a reasonable default. It is an arbitrary value chosen to > ensure that coded frames can't be too far apart before an out-of-order > append error is signalled.
Changes committed. https://dvcs.w3.org/hg/html-media/rev/f7f2b7226543 - Made sample rate & channel layout conversion only occur when necessary. - Specified starting & ending gains for the fades. I think this is a little easier to understand then just specifying a slope since the slope units aren't exactly clear. The gain & time ranges are now specified so implementers should have no problem computing the appropriate slope for their needs.