search

victronenergy / dbus-ble-sensors

6 stars 4 forks source link

Adding support for Mopeka tank sensors #3

Closed robsweet closed 1 year ago

robsweet commented 2 years ago

Regarding testing, development was done with a Cerbo GX and Mopeka Pro LPG sensor. Mr.Ghostrider also helped test with Venus on a Raspberry Pi and said that looked fine and that Ruuvi sensors still worked properly. I don't have a Mopeka Pro sensor for water tanks so I can't test that but the logic to calculate the level is smart enough to know which kind of sensor is being reported so it can use the right coefficient values for the calculation.

robsweet commented 2 years ago

Changed MopekaSensorVersion to MopekaSensorTypeId at request of @mpvader to better align with Victron naming conventions.

mansr commented 2 years ago

Is the data format documented anywhere? I'm trying to make sense of the code, and that would be much easier if I knew what I was actually dealing with.

robsweet commented 2 years ago

Is the data format documented anywhere? I'm trying to make sense of the code, and that would be much easier if I knew what I was actually dealing with.

I've got a doc about the format but I've been asked not to make it public. Please email me directly at Rob@LDG.net and I'll send it to you.

mansr commented 1 year ago

Some version of this was merged a long time ago.