search

core-wg / senml-spec

Specification for SenML
22 stars 4 forks source link

Units fixes #6

Closed cabo closed 8 years ago

cabo commented 8 years ago

This should now consider #4 and #5. There are no perfect solutions for these, but the solutions chosen should be close.

fluffy commented 8 years ago

I don't agree wth some of this but agree with a bunch. I took carsten's changes for all the parts I agree with and put them in

https://github.com/core-wg/senml-spec/pull/9

fluffy commented 8 years ago

I might suggest we pull that #9 PR and then look at what is left (on a bunch of them it is simply if there should be an * suggesting don't use or not) then we can rebase this PR and see what is left to resolve.

cabo commented 8 years ago

Oops, I just changed my PR to react to your comments before seeing #9

cabo commented 8 years ago

Pushed another version that has both / and % for the same thing, where % is marked with a *

akeranen commented 8 years ago

Now, instead of delete, it's "not recommend" and mentions kg as exception. I think this is good to merge.