Re: [rtcweb] Review of draft-ietf-rtcweb-fec-06

[Magnus Westerlund <[e-mail]>]

Hi,

Sorry, for not having gotten to this until now. The changes looks good. 
However, I noticed that the 26.114 references references the 13.3 
version. I looked at the latest 14.4 which appears to have the content 
referenced at the stated locations. However, if there has been changes 
to any related text I haven't compared. Even if we would choose to stay 
with release 13, then the version may still be changed to 13.5 which is 
the latest release 13 version.

Cheers

Magnus


Den 2017-09-01 kl. 23:41, skrev Justin Uberti:
> Magnus, did you have a chance to take a look at this? I think this is 
> the sole actionable remaining issue for this document.
>
> On Sun, Aug 27, 2017 at 1:25 PM, Justin Uberti <[e-mail] 
> <mailto:[e-mail]>> wrote:
>
>     PTAL at https://github.com/juberti/draughts/pull/89
>     <https://github.com/juberti/draughts/pull/89>
>
>     On Fri, Jul 21, 2017 at 12:51 AM, Magnus Westerlund
>     <[e-mail]
>     <mailto:[e-mail]>> wrote:
>
>
>
>         Den 2017-07-20 kl. 08:25, skrev Justin Uberti:
>>         Thanks for your comments. Regarding #2, the document
>>         currently says the following:
>>
>>             Since use of FEC always causes redundant data to be transmitted, this
>>             will lead to less bandwidth available for the primary encoding when
>>             in a bandwidth-constrained environment.
>>         Do you think this is insufficient?
>
>         Yes, I think it should be more explicit that primary + repair
>         need to be within the bandwidth limits set by congestion
>         control, etc.
>
>         /Magnus
>
>>
>>         On Sun, Jul 16, 2017 at 2:40 AM, Magnus Westerlund
>>         <[e-mail]
>>         <mailto:[e-mail]>> wrote:
>>
>>             Hi,
>>
>>             I was reviewing -06 to ensure that all issues are
>>             resolved. I did notice the following issues.
>>
>>
>>             1. Section 4.2, makes a normative refernces to TS.26114,
>>             which is listed as informative refences in the reference
>>             list. Please move it to the normative part.
>>
>>             2. Section 8. I am missing the statement that primary
>>             encoding + FEC needs to keep within the available
>>             bandwidth as determined by the congestion control or
>>             other bitrate adaptation mechnisms. Thus, likely
>>             resulting in that the primary encoding bitrate needs to
>>             be reduced to fit the FEC.
>>
>>             Otherwise the document looks good.
>>
>>             Cheers
>>
>>             Magnus Westerlund
>>
>>             ----------------------------------------------------------------------
>>             Media Technologies, Ericsson Research
>>             ----------------------------------------------------------------------
>>             Ericsson AB                 | Phone [phone]
>>             <tel:%2B46%2010%207148287>
>>             Torshamnsgatan 23           | Mobile [phone]
>>             <tel:%2B46%2073%200949079>
>>             SE-164 80 Stockholm, Sweden | mailto:
>>             [e-mail]
>>             <mailto:[e-mail]>
>>             ----------------------------------------------------------------------
>>
>>
>>
>
>         -- Magnus Westerlund
>         ----------------------------------------------------------------------
>         Media Technologies, Ericsson Research
>         ----------------------------------------------------------------------
>         Ericsson AB | Phone [phone]
>         <tel:+46%2010%20714%2082%2087>Torshamnsgatan 23 | Mobile +46
>         73 0949079 <tel:+46%2073%20094%2090%2079> SE-164 80 Stockholm,
>         Sweden | mailto: [e-mail]
>         <mailto:[e-mail]>
>         ----------------------------------------------------------------------
>
>
>

-- 

Magnus Westerlund

----------------------------------------------------------------------
Media Technologies, Ericsson Research
----------------------------------------------------------------------
Ericsson AB                 | Phone  [phone]
Torshamnsgatan 23           | Mobile [phone]
SE-164 80 Stockholm, Sweden | mailto: [e-mail]
----------------------------------------------------------------------