Re: [core] Benjamin Kaduk's Discuss on draft-ietf-core-senml-more-units-04: (with DISCUSS and COMMENT)

Carsten Bormann <cabo@tzi.org> Wed, 19 February 2020 18:07 UTC

Return-Path: <cabo@tzi.org>
X-Original-To: core@ietfa.amsl.com
Delivered-To: core@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0938112083F; Wed, 19 Feb 2020 10:07:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 oBCSn1Yu696K; Wed, 19 Feb 2020 10:07:22 -0800 (PST)
Received: from gabriel-vm-2.zfn.uni-bremen.de (gabriel-vm-2.zfn.uni-bremen.de [134.102.50.17]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8B5331208B3; Wed, 19 Feb 2020 10:07:21 -0800 (PST)
Received: from client-0044.vpn.uni-bremen.de (client-0044.vpn.uni-bremen.de [134.102.107.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gabriel-vm-2.zfn.uni-bremen.de (Postfix) with ESMTPSA id 48N5KJ4BgPzyRm; Wed, 19 Feb 2020 19:07:16 +0100 (CET)
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 13.0 \(3608.60.0.2.5\))
From: Carsten Bormann <cabo@tzi.org>
In-Reply-To: <20200219170126.GE11645@kduck.mit.edu>
Date: Wed, 19 Feb 2020 19:07:15 +0100
Cc: The IESG <iesg@ietf.org>, draft-ietf-core-senml-more-units@ietf.org, Jaime Jimenez <jaime@iki.fi>, core-chairs@ietf.org, core@ietf.org
X-Mao-Original-Outgoing-Id: 603828435.299191-de11cc49f0886353e5fb78c322b16ce4
Content-Transfer-Encoding: quoted-printable
Message-Id: <3B006531-021E-46AE-BCD9-DDD298EAEDCE@tzi.org>
References: <158207175238.13976.4748538225663851323.idtracker@ietfa.amsl.com> <89E1D428-FF1B-4B42-BF1A-2BEF9B2C0B1B@tzi.org> <20200219170126.GE11645@kduck.mit.edu>
To: Benjamin Kaduk <kaduk@mit.edu>
X-Mailer: Apple Mail (2.3608.60.0.2.5)
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/SWnqA6Q2LpCVvJ_HJcgTzgrJnu4>
Subject: Re: [core] Benjamin Kaduk's Discuss on draft-ietf-core-senml-more-units-04: (with DISCUSS and COMMENT)
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Constrained RESTful Environments \(CoRE\) Working Group list" <core.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/core>, <mailto:core-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/core/>
List-Post: <mailto:core@ietf.org>
List-Help: <mailto:core-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/core>, <mailto:core-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Feb 2020 18:07:25 -0000

Hi Ben,

> On 2020-02-19, at 18:01, Benjamin Kaduk <kaduk@mit.edu> wrote:
> 
> Hi Carsten,
> 
> Thanks for the updates; the changes in the -05 look good with just one
> exception.  That, and some other comments, inline.
> 
>> 
>>> ----------------------------------------------------------------------
>>> COMMENT:
>>> ----------------------------------------------------------------------
>>> 
>> 
>>> Section 3
>>> 
>>>  o  The Byte.  [IEC-80000-13] defines both the bit (item 13-9.b) and
>>>     the byte (item 13-9.c, also called octet) as alternative names for
>>>     the coherent unit one for the purpose of giving storage capacity
>>>     and related quantities.  While the name octet is associated with
>>> 
>>> nit: is "one" misplaced, here?
>> 
>> The coherent unit actually is “one”; see Section 3.8, Note 3, and Section 3.12, Note 4 in ISO 80000-1 as cited above.
>> To make this easier to read, we could place it in quotes, except it is not a string, it is really the number 1.
> 
> I guess I kind of gave it away that I didn't follow the reference, here :)
> It's probably too verbose to try something like "the coherent unit used for
> dimensionless quantities and indicated by the numeral one".

Maybe not the whole thing, but I like "the coherent unit used for
dimensionless quantities” instead of "coherent unit one”.  People who understand the latter will always understand the former.

>>> Section 4
>>> 
>>>  o  scale, offset: two rational numbers, expressed in decimal
>>>     (optionally, with a decimal exponent given) or as a fraction
>>>     divided by a "/" character.
>>> 
>>> nit: doesn't "a fraction divided by a '/' character" involve two '/'
>>> characters?  That is, "1/2" is a fraction, so "1/2 divided by a '/'
>>> character" would be like "1/2/" or "1/2/2" or something nonsensical like
>>> that.
>> 
>> “divided” as in “separated into two halves”, i.e., “1/2” is divided by a “/“ into a 1 and a 2.  Maybe we need better phrasing here.
> 
> Perhaps, "a fraction represented using a '/' character to separtae
> numerator and denominator"?

That works very well.

>> […]
>> I have turned this into
>> 
>> (or a similar
>> mechanism visiting IANA less frequently)
> 
> This is better, but I think that "less frequently" is still an implied
> comparison, and the comparison against software-update is more local than
> the (intended) comparison against the generic/abstract "usual frequency" or
> "every time an instance of the system starts up or checks".  I might
> suggest "a similar mechanims that only visits IANA infrequently".

I think the criticism of visiting IANA from every instance should be clear here, so it seems to me people should be able to relate the “visit IANA less frequently” to the “download … from IANA frequently” above.  (I’m not sure what infrequently means; I think once an hour would be fine for a major software package that then distributes updates via software update mechanism.)
Maybe “that leads to less frequent IANA visits” is even clearer?
(Because it’s not the software update mechanism itself that visits IANA.)

[…]

Proposed updates in
https://github.com/core-wg/senml-more-units/commit/ebb13e358e59327d243f4dff617c9b19b5168cd9

Grüße, Carsten