Re: [core] draft-ietf-core-senml-more-units-00

Jaime Jiménez <jaime@iki.fi> Fri, 13 September 2019 09:12 UTC

Return-Path: <jaime@iki.fi>
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 417BB120255 for <core@ietfa.amsl.com>; Fri, 13 Sep 2019 02:12:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.82
X-Spam-Level:
X-Spam-Status: No, score=-1.82 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_NEUTRAL=0.779, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=messagingengine.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 CsfUJazZWPXV for <core@ietfa.amsl.com>; Fri, 13 Sep 2019 02:12:37 -0700 (PDT)
Received: from wout1-smtp.messagingengine.com (wout1-smtp.messagingengine.com [64.147.123.24]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 156EC1208AA for <core@ietf.org>; Fri, 13 Sep 2019 02:12:37 -0700 (PDT)
Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id 3E6FE4D3; Fri, 13 Sep 2019 05:12:36 -0400 (EDT)
Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Fri, 13 Sep 2019 05:12:36 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=VnLfa10CcDV5dUhf0LNG0MDbmrRrOX587VQpZu/RH sQ=; b=iHP6TDgt1FhBcmqoiQFBV4Owr7OVqwvoI9IrhZjoMpoOPz/oiLf0hD2Np GEUDMBNHYsNB8hjIMmWGqYXc8bKKfOcjSRP3VAvxOPVOergwrvxotL144Pb8Pd8G LyWlcXosQgC6nTQ9y7+2aLvFYGsGDn7lYZWW2RJ6C+p5XtW5RN9904PBAjEhiYYE /AUgnQTciPwoLc1K74sbzPnX16ui+ci3zyMP7j3B/72wRIq95gyN8ksQ/J3v3Q3X 1GBZizZgvZtQI92eSDZvd7htrs85iug70Jh9lCpVRZ0q1m/2XdTWGw/ib//S0FmY wK9tc4qLD4nM6Y575OfxreY/1z/vg==
X-ME-Sender: <xms:g117XZY3yjP3EZ1YPPHhg1YHC0XHW8UnUXGwSkDe5RRWzHe1IaWJJg>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedufedrtdejgdduvdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvuffkfhggtggugfgjfgesthekredttderudenucfhrhhomheplfgrihhm vgculfhimhornhgviicuoehjrghimhgvsehikhhirdhfiheqnecuffhomhgrihhnpehivg htfhdrohhrghenucfkphepkeelrdduieeirdegledrvdegfeenucfrrghrrghmpehmrghi lhhfrhhomhepjhgrihhmvgesihhkihdrfhhinecuvehluhhsthgvrhfuihiivgeptd
X-ME-Proxy: <xmx:g117XSb2vgxdDAUX3dRGLkmPot7ub2NtUIDxg8IFlChzVd0GQcYPuQ> <xmx:g117XQSnt49rbHlpUHNieEjEYmPgjSjDix3s2asImZRPNRaYoAlKaw> <xmx:g117Xf3Q08Iyh9nuw1amdflvHmsKSkAFkFKz-NSWUDsZGuy0-LiR8g> <xmx:g117XcoWAyJvG4pM3ovpWMAiSGLSdasdnfFB_b5fYFBA1pIAJEV_QQ>
Received: from EMB-918HFH01 (89-166-49-243.co.dnainternet.fi [89.166.49.243]) by mail.messagingengine.com (Postfix) with ESMTPA id 7E60780062; Fri, 13 Sep 2019 05:12:33 -0400 (EDT)
Date: Fri, 13 Sep 2019 12:12:31 +0300
From: Jaime Jiménez <jaime@iki.fi>
To: Carsten Bormann <cabo@tzi.org>
Cc: Hannes Tschofenig <Hannes.Tschofenig@arm.com>, "core@ietf.org WG" <core@ietf.org>
Message-ID: <20190913091227.d46rkqpfi4x43hxc@EMB-918HFH01>
References: <AM0PR08MB5345AEFD7FDDBE0CB9192787FAB00@AM0PR08MB5345.eurprd08.prod.outlook.com> <CF31A377-036F-4F9C-8AB3-C79748AEC10C@tzi.org> <20190913061538.qmls3fk7indtpqd3@EMB-918HFH01>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20190913061538.qmls3fk7indtpqd3@EMB-918HFH01>
User-Agent: NeoMutt/20180716
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/QZQvqUMF47-oJIf30W3oi4Cw2Qs>
Subject: Re: [core] draft-ietf-core-senml-more-units-00
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: Fri, 13 Sep 2019 09:12:40 -0000

Hi Carsten,

here is the review for the draft.

## General comments

- The Secondary Registry table is probably lacking a description column
  similar to the main SenML registry.

- It looks like "ppm" is not derived from any main SenML unit. As the
  secondary registry is explicity for derived units, shouldn't ppm
  be then in the main SenML registry?   

- There is an open comment on whether the text should update RFC 8428,
  IMO the answer is yes but maybe the authors of both documents and the
  group should decide.

## Editorial

- P4: Says "four", however including the reference there would be five
  columns. 

- P5: "Table 2" is missing the caption.

- P6,P7: Wouldn't the references to IEC and IEEE be informative rather than
  normative?

- P7: IMHO for informational purposes it would help a lot to have references
  to the right locations in BIPM and the IEC, which are referenced themselves in [RS]. 

## Other

- The document indicates that new entries will be added through "Expert
  Review". Often it is the case that little documentation is provided
  when new entries are given to IANA and, since we should avoid
  confusion, I would like to know if we will have specific experts
  versed in the topics of Units to review the entries.

Ciao!


On Fri, Sep 13, 2019 at 09:15:39AM +0300, Jaime Jiménez wrote:
> Hi,
> 
> I'll try to provide a review in the coming days. 
> 
> Ciao!
> 
> On Fri, Sep 13, 2019 at 07:07:56AM +0300, Carsten Bormann wrote:
> > Hi Hannes,
> > 
> > Quick answer from my vacation:
> > The discussion we had during WG adoption raised a couple minor points, but nothing we couldn't handle during a WGLC. So if Jaime can do a chair's review soon, we could WGLC the document afterwards, say, until about Sep 30. 
> > Deg can be registered right away, but adding the second table requires iesg approval; so we should submit to iesg by, say, Oct 1 and could be approved by the end of the month, barring surprises. This is optimistic, but not unrealistic at all. 
> > 
> > Sent from mobile, sorry for terse
> > 
> > > On 12. Sep 2019, at 18:27, Hannes Tschofenig <Hannes.Tschofenig@arm.com> wrote:
> > > 
> > > Hi Carsten, Hi all,
> > >  
> > > When do you expect draft-ietf-core-senml-more-units-00 to be completed?
> > >  
> > > Ciao
> > > Hannes
> > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
> > > _______________________________________________
> > > core mailing list
> > > core@ietf.org
> > > https://www.ietf.org/mailman/listinfo/core
> 
> > _______________________________________________
> > core mailing list
> > core@ietf.org
> > https://www.ietf.org/mailman/listinfo/core
> 
> _______________________________________________
> core mailing list
> core@ietf.org
> https://www.ietf.org/mailman/listinfo/core