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

Bence Béky <notifications@github.com> Wed, 16 October 2019 18:02 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 D524B120827 for <quic-issues@ietfa.amsl.com>; Wed, 16 Oct 2019 11:02:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.382
X-Spam-Level:
X-Spam-Status: No, score=-6.382 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_24=1.618, 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 JfBFPC-wDLKS for <quic-issues@ietfa.amsl.com>; Wed, 16 Oct 2019 11:02:06 -0700 (PDT)
Received: from out-24.smtp.github.com (out-24.smtp.github.com [192.30.252.207]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1CC75120958 for <quic-issues@ietf.org>; Wed, 16 Oct 2019 11:02:06 -0700 (PDT)
Date: Wed, 16 Oct 2019 11:02:05 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1571248925; bh=F8fXDnnV5/bCEgCADxGGpBErBKfaGmxGiDPMBbwRWvc=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=z3ygWlaG5svLYabdFSDhroGs5CBBiTD8S0adMhL3ED3v2qwmTwS2RdZ291ISkgwiM b0V7Oxy2t4p/KAPvqP4MasiBRbD2eyB1Df1uKmKqHayzSACGmpmPwSsXpHUftZAF2m R9Yrqx3L85a72gKhXoXMF6AiaPSL/pMvqmnLC0+Q=
From: =?UTF-8?B?QmVuY2UgQsOpa3k=?= <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+AFTOJK4GLSSVJ57ELSRHXVF3WSN23EVBNHHBYU2QL4@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/c542821859@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_5da75b1d4e8e0_13d53fcd140cd96c123174"; 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/_rHd-dfNVeirThW0ZzMITauBAa4>
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 18:02:08 -0000

> In a one-pass encoder, Base is the Insert Count when the encoder begins encoding the block. In a two-pass encoder, I assume most folks set Base = RIC. So I think it logically is a count, but lets put aside whether to rename it since that's not happening in this PR.
> 
> For now, I say remove RIC from the figure, but leave the arrow where it is.

Sounds good to me.  Let me add text specifying the Base value because it is not clear for those not familiar with the "indices point between elements" mental model or would be confused by the element with a given index being the one not on the right of the arrow but on the left.

-- 
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-542821859