Affirmatech / MeshSense

MeshSense directly connects to your Meshtastic node via Bluetooth or WiFi and continuously provides information to assess the health of your mesh network.
https://affirmatech.com/meshsense
GNU General Public License v3.0
46 stars 6 forks source link

BatteryLevel 101% #15

Closed karamo closed 1 month ago

karamo commented 1 month ago

The "batteryLevel": 101 is a magic number and means USB powered

scruplelesswizard commented 1 month ago

Meshtastic Firmware Reference

mblauser commented 1 month ago

What suggestion do you have for representing this? Perhaps replacing the battery% with a 🔌 icon/emoji?
The voltage is still reported while on external power, so that could still remain as-is.

karamo commented 1 month ago

Yes, I prefere a symbol for this case. And in the Voltage-field a "-". A negative voltage ist sensless. The voltage value ist undefined in the FW.

N3FJP commented 1 month ago

Hi Guys, thanks so much for your chiming in and supporting MeshSense! Two thoughts...

Thanks again!

mblauser commented 1 month ago

I tend to agree with not using 🔌 as it could be inappropriate for solar nodes. It is easier and most scalable to use a unicode symbol, so perhaps the use of⚡️⚡︎ or ⌁ is better suited.

mblauser commented 1 month ago

I have introduced draft pull request #17 replacing 101% with a ⚡︎ unicode symbol. If it seems agreeable to the majority, I'll close the draft and queue it for proposed adoption.

image
Soltares commented 1 month ago

Closing as #17 is included in latest release