Re: [xrblock] Fw: I-D Action: draft-ietf-xrblock-rtcp-xr-jb-08.txt

[Qin Wu <[e-mail]>]

Hi,Kevin:
Here is the update to address your concern.
http://www.ietf.org/internet-drafts/draft-ietf-xrblock-rtcp-xr-jb-10.txt
The diff is
http://www.ietf.org/rfcdiff?url2=draft-ietf-xrblock-rtcp-xr-jb-10

Regards!
-Qin


  ----- Original Message ----- 
  From: Qin Wu 
  To: Kevin Gross 
  Cc: Alan Clark ; xrblock ; Romascanu, Dan (Dan) ; Shida Schubert 
  Sent: Tuesday, April 02, 2013 11:17 AM
  Subject: Re: Fw: I-D Action: draft-ietf-xrblock-rtcp-xr-jb-08.txt


  Hi, Kevin:
  Thank for your response, see my reply inline below.

  Regards!
  -Qin
    ----- Original Message ----- 
    From: Kevin Gross 
    To: Qin Wu 
    Cc: Alan Clark ; xrblock ; Romascanu, Dan (Dan) ; Shida Schubert 
    Sent: Tuesday, April 02, 2013 5:12 AM
    Subject: Re: Fw: I-D Action: draft-ietf-xrblock-rtcp-xr-jb-08.txt


    Hi Qin,


    Responses below.


    Kevin



    On Wed, Mar 20, 2013 at 6:45 AM, Qin Wu <[e-mail]> wrote:

      Hi,Kevin:
      Thank for your quick checking.
      Please see my reply inline.

      Regards!
      -Qin
        ----- Original Message ----- 
        From: Kevin Gross 
        To: Qin Wu 
        Cc: Alan Clark ; [e-mail] ; Romascanu, Dan (Dan) ; Shida Schubert 
        Sent: Sunday, March 17, 2013 6:10 AM
        Subject: Re: Fw: I-D Action: draft-ietf-xrblock-rtcp-xr-jb-08.txt


        I said my contributions were rough and it doesn't look like they were improved much in the new revision.


        Section 3 


        "The delay applied to
   packets that arrive at their expected time is known as the Nominal
   Delay and this is equivalent to the late edge."Nominal delay is not equivalent to late edge. [Qin]: Please refer to example provide by Alan in Jan 16,2013 when clarifing to Roni on what nominal delay means.http://www.ietf.org/mail-archive/web/xrblock/current/msg01090.htmlOne sentence in Alan's example I highlighted here is:"So the nominal delay is the time difference/ buffer size difference between the on-time insertion point and the point at which packets are read out and decoded."So the nominal delay is equivalent to the time difference between the expected arrival time and the time when a late arriving packet is consumed.or we can say nominal delay is equivalent to late edge, what am I missing?

    If we are not including decoder delay in this discussion, I think you're right about this. I'm now too close to this to be able to tell whether it is explained clearly. Hopefully others will review.

  [Qin]: Okay, I agree with you that the decoder delay should not be taken into account. So the sentence I am referrring will be rephrased like this:
  "
  So the nominal delay is the time difference/ buffer size difference between the on-time packets insertion point and the point at which packets are read out.
  " 

Section 3.2"The fixed jitter buffers have a fixed size and the packets leaving
   the jitter buffer have a constant delay."There is just one jitter buffer and we need a bit more elaboration on how a fixed jitter buffer is used, its limitations and how maximum delay and nominal delay are reported for it. [Qin]: I believe these have been explained in the Definition of Fields in Jitter Buffer Metrics Block of section 4.2.

    I don't see it. Section 4.2 is the longest section. Fixed jitter buffer term is used there but never defined or elaborated on. In any case, this does not address my concern about singular vs. plural buffers and subject-verb disagreement here.

  [Qin]: You are right. It should be singular. To address your concern, I propose the following change to section 3.2:
  OLD TEXT
  "
Section 3.2"The fixed jitter buffers have a fixed size and the packets leaving
   the jitter buffer have a constant delay.""
  NEW TEXT:
  "
Section 3.2A fixed jitter buffer lacks provision to track network condition and  has a fixed size and packets leaving the jitter buffer.have a constant delay. For fixed jitter buffer implementation, the maximum delay is set to a constant value corresponding to the fixed size of the jitter buffer. The nominal delay is set to a constant value corresponding to the packets that arrive at their expected arrival time.

  "
Section 3.3"An adaptive jitter buffer have variable size and variable delay."Fix grammar. Change "and" to "or" or "and/or". [Qin]: Agree, thanks.

    Just to be clear, those are two separate complaints. There are several instances of subject-verb disagreements. We can say those are just editorial issues but I think they need to be addressed here because it it is not always clear from context whether singular or plural is intended.

  [Qin]:My fault. I think it should be singular and will fix this.