Re: [quicwg/base-drafts] QPACK [editorial] update RIC and Base on figures (#2935)

Bence Béky <notifications@github.com> Wed, 16 October 2019 13:33 UTC

Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D89C81200DE for <quic-issues@ietfa.amsl.com>; Wed, 16 Oct 2019 06:33:19 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.596
X-Spam-Level:
X-Spam-Status: No, score=-6.596 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_28=1.404, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id zOjL9_9AN8th for <quic-issues@ietfa.amsl.com>; Wed, 16 Oct 2019 06:33:18 -0700 (PDT)
Received: from out-23.smtp.github.com (out-23.smtp.github.com [192.30.252.206]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 4D9521200D5 for <quic-issues@ietf.org>; Wed, 16 Oct 2019 06:33:18 -0700 (PDT)
Received: from github-lowworker-f045d1f.ac4-iad.github.net (github-lowworker-f045d1f.ac4-iad.github.net [10.52.19.54]) by smtp.github.com (Postfix) with ESMTP id A7FAA6611F1 for <quic-issues@ietf.org>; Wed, 16 Oct 2019 06:33:17 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1571232797; bh=XQFy983nqB3Mdoh8to8D5G0k0wMiQ8Lw6R430rJ25yY=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=Xps7oCWV/QghI/GD5SeuKAd6tvDaZQG97oYnZknbNIyiK1430y8+6RAMV5p0tG6tq jiHPMr1lvwgr0frgIdUNpCqhlyxVBheJMoyK2n1Jvwbmhx5J1Ooj+90bhKhkyaiO85 iiZDOkSb69FNI3YxFwokbMVYkhVLsk7AaKighoA0=
Date: Wed, 16 Oct 2019 06:33:17 -0700
From: =?UTF-8?B?QmVuY2UgQsOpa3k=?= <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK3MASXPGPJDR6FTLFF3WROK3EVBNHHBYU2QL4@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2935/c542702765@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2935@github.com>
References: <quicwg/base-drafts/pull/2935@github.com>
Subject: Re: [quicwg/base-drafts] QPACK [editorial] update RIC and Base on figures (#2935)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5da71c1d98c3d_7b253fdaf1acd96819413b"; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: bencebeky
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/X3ugmbeiLbJ7Qt0_fBoXivP6BFI>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 16 Oct 2019 13:33:20 -0000

Only the author of this PR and editors of the spec ever commented on this editorial issue.  This indicates to me that this issue is insignificant.  I encourage any of the editors to just make a decision on where Base should point or if it should appear on the drawing at all.  I'm happy to implement that so that we can cross this PR off the list.

If the decision is that Base should point between two elements, or that it should be removed from the drawing, then I suggest adding text specifying that Base = n-2.

By the way it just occurred to me that RIC does not belong to this diagram at all.  In fact it is possible that a header block does not reference the most recently inserted element, in which case RIC has a different value than currently indicated.

In my mental model Base is neither an index nor a count.  It is an integer that the encoder sends to the decoder, which is then used for arithmetics described in the spec to convert between absolute and relative indices.  I'm not convinced that renaming it to Base Count would increase clarity.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/2935#issuecomment-542702765