Add support for more IPSO objects. - Githubissues

TheThingsNetwork / go-cayenne-lib

CayenneLPP in Go

MIT License

4 stars 6 forks source link

Add support for more IPSO objects. #12

Open ofauchon opened 2 years ago

ofauchon commented 2 years ago

Hi. @johanstokking , @calvernaz.

I'm using your library in my Lorawan connected devices (TinyGo based code)

Now, I need more types of measurements (power related : voltage, intensity power) for monitoring power inverters.

I can see these types of objects exists in IPSO object Registry

https://technical.openmobilealliance.org/OMNA/LwM2M/LwM2MRegistry.html https://github.com/OpenMobileAlliance/lwm2m-registry

ID=3316 : Voltage/Volts
ID=3317 : Current/Amps
ID=3318 : Frequency/Hz
ID=3328 : Power/Watt

Do you think I can propose a PR for supporting these extra measurements:

AddVoltage()
AddCurrent()
AddFrequency()
AddPower()

Maybe we could also think about a more generic API like:

AddCustomMeasurement(IPSO_Object, Value)

Thanks for your help,

Olivier

johanstokking commented 2 years ago

Hmm, I'm not sure if defined IPSO objects should automatically be used in CayenneLPP. We don't know the data size or the data resolution. Therefore we follow https://community.mydevices.com/t/cayenne-lpp-2-0/7510.

I would suggest filing this with myDevices.