Re: [Cellar] I-D Action: draft-ietf-cellar-ffv1-15.txt

[Spencer Dawkins at IETF <[e-mail]>]

Dear Cellar,

On one point,

On Thu, Jun 25, 2020 at 11:25 AM Murray S. Kucherawy <[e-mail]>
wrote:

> Hi, thanks for this.  I'm looking now only at the diff between -13 (the
> last version I looked at) and this one.
>
> First and foremost, this is mostly better.  Kudos for putting the work in.
>
> Hooray, most of those "type" columns are gone!  But, alas, not all of
> them.  They're still present in Sections 4.1 and below.  I counted at least
> seven of them just in the diff I'm reviewing.  As before, they should also
> be removed, or at least explained.
>
> A minor point: Prior to Section 3.8.2.3, it looks like all of the examples
> are in C.  After that, they switch to pseudo code.  Any reason we can't be
> consistent?
>
> I suggest that the prose below Table 8, which is a mix of pseudocode and
> English, would be better broken apart.
>
> Chairs: RFC 4732 is a downref; please make sure the shepherd calls this
> out in the writeup.
>

This means that RFC 4732 is not a standards-track document, while our draft
is standards-track. That's fine, but we need to point that out when the
document is sent for IETF Last Call, so the shepherd (or one of the working
group chairs) should mention this in the "Other Points" section of
https://datatracker.ietf.org/doc/draft-ietf-cellar-ffv1/shepherdwriteup/.

Best,

Spencer


>
>
> There are a few code expressions that would benefit from being wrapped
> differently.  For example:
>
> (chroma_planes == 1 && (p == 1 || p == 2)) ? ceil(slice_pixel_height
> / (1 << log2_v_chroma_subsample)) : slice_pixel_height
>
> Maybe this?
>
> chroma_planes == 1 && (p == 1 || p == 2)
>   ? ceil(slice_pixel_height / (1 << log2_v_chroma_subsample))
>   : slice_pixel_height
>
> -MSK
>
> On Tue, Jun 23, 2020 at 12:26 PM Spencer Dawkins at IETF <
> [e-mail]> wrote:
>
>> Hi, Murray,
>>
>> We just confirmed during the Cellar virtual meeting that -15 IS actually
>> ready for you to check and confirm that we've addressed the AD comments on
>> this draft.
>>
>> Best,
>>
>> Spencer
>>
>> On Tue, Jun 23, 2020 at 2:12 PM <[e-mail]> wrote:
>>
>>>
>>> A New Internet-Draft is available from the on-line Internet-Drafts
>>> directories.
>>> This draft is a work item of the Codec Encoding for LossLess Archiving
>>> and Realtime transmission WG of the IETF.
>>>
>>>         Title           : FFV1 Video Coding Format Version 0, 1, and 3
>>>         Authors         : Michael Niedermayer
>>>                           Dave Rice
>>>                           Jerome Martinez
>>>         Filename        : draft-ietf-cellar-ffv1-15.txt
>>>         Pages           : 53
>>>         Date            : 2020-06-23
>>>
>>> Abstract:
>>>    This document defines FFV1, a lossless intra-frame video encoding
>>>    format.  FFV1 is designed to efficiently compress video data in a
>>>    variety of pixel formats.  Compared to uncompressed video, FFV1
>>>    offers storage compression, frame fixity, and self-description, which
>>>    makes FFV1 useful as a preservation or intermediate video format.
>>>
>>>
>>> The IETF datatracker status page for this draft is:
>>> https://datatracker.ietf.org/doc/draft-ietf-cellar-ffv1/
>>>
>>> There are also htmlized versions available at:
>>> https://tools.ietf.org/html/draft-ietf-cellar-ffv1-15
>>> https://datatracker.ietf.org/doc/html/draft-ietf-cellar-ffv1-15
>>>
>>> A diff from the previous version is available at:
>>> https://www.ietf.org/rfcdiff?url2=draft-ietf-cellar-ffv1-15
>>>
>>>
>>> Please note that it may take a couple of minutes from the time of
>>> submission
>>> until the htmlized version and diff are available at tools.ietf.org.
>>>
>>> Internet-Drafts are also available by anonymous FTP at:
>>> ftp://ftp.ietf.org/internet-drafts/
>>>
>>>
>>> _______________________________________________
>>> Cellar mailing list
>>> [e-mail]
>>> https://www.ietf.org/mailman/listinfo/cellar
>>>
>>