TheThingsNetwork / go-cayenne-lib

CayenneLPP in Go
MIT License
3 stars 6 forks source link

add support for voltage, current, frequency and energy #9

Closed selvakn closed 6 years ago

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

johanstokking commented 6 years ago

@selvakn apologies for the late follow up and thanks for the clean PR.

Is this CayenneLPP 2.0?

selvakn commented 6 years ago

@johanstokking no worries. This is not part of CayenneLPP yet, but based on IPSO, which cayenneLPP is based on. Im trying to reach Eric Ptak from Cayenne to discuss about the right precision and have it added there.

Meanwhile, having it in TTN would help my app.

johanstokking commented 6 years ago

Thanks for clarifying. We are keeping this repository consistent with CayenneLPP's specs for now. Until myDevices publishes a new release, we should not be rolling our own fork.

aparcar commented 3 years ago

It's been some time, any update on Cayenne supporting more IPSO formats? Other libraries moved on and added them, including a TTN compatible decoders script.

johanstokking commented 3 years ago

That's the problem with CayenneLPP really; forks with new fields.

We keep https://github.com/myDevicesIoT/CayenneLPP as authoritive source.