[core] Review of draft-ietf-core-senml-versions-01

[Jaime Jiménez <[e-mail]>]

Dear authors,

first of all I'd like to apologise for the delay when providing the review.

While I did not find any mistakes, there are few additions to the document that *could* help clarify it for the benefit of the readers:

- Mention somewhere in the text that the maximum amount of features is 53 and that therefore the highest version number is 2**53-1 in decimal notation. Therefore readers should not expect the usual linear semantic versioning for each version (1,2,3,4...) but rather (10, 26, 42, 58...).

- Give an example of the current bitmap -for example in a double or float representation- providing all features that are currently supported for bver=26.

00000000 00000000
00000000 00000000
00000000 00000000
00000000 00011010

- The limitation to 53 versions in total as well as potential feature deprecation was already discussed in various IETFs and neither authors nor the group see that as a limitation. I agree with that providing that expert review is conservative when adding new features. Potentially we could also think about bundling some future features if they are related before publication; but it is very hard to forecast that.

Note that, ahead of the shepherd writeup, it would be good to document existing implementations out there.

Editorial:

"Network Working Group" should be CoRE.

independently selectable "features"
Use of quotes for features seems unnecessary, as those literally are features.                       
   
Ciao!
-- 
Jaime Jiménez