meshtastic / Meshtastic-Android

Android application for Meshtastic
https://meshtastic.org
GNU General Public License v3.0
748 stars 217 forks source link

[Bug]: Device Telemetry - Power metrics: current values #1288

Closed joneyes closed 1 month ago

joneyes commented 1 month ago

Contact Details

jon.eyes@duncreggan.net

What happened?

Mutliple current values displayed - only started on this version. node has a single channel INA 219 current sensor used to monitor battery charge/discharge current. The same symptom occurs when data is received via the mesh, or directly connected to the RAK via BLE image

App Version

2.4.17

Phone

Samsung s22 Ultra, Android 15, One UI 6.1

Device

RAK4631, with INA219 current sensor board connected on I2C

Firmware

2.5.3.a70d5ee

Relevant log output

No response

Code of Conduct

andrekir commented 1 month ago

all data comes from the radio. power metrics channel 2 & 3 were added in .17 and the app displays that data as received.

joneyes commented 1 month ago

Hmm, OK. I have the following observations then.

There is a single ina219 sensor connected, with a single channel. It's connected low side. Prior to this change, the measured idle current was about 11ma, which from memory, reflected what the board was drawing when measured with a multimeter. I now see two current values, neither of which reflect the load current, either individually or as a sum?

b8b8 commented 1 month ago

Try bridging A0 to hard set the i2c address. I had to recently do this to an INA3221 to get it to read correctly. Sometimes the default address isnt picked up correctly (im guessing). image

image

joneyes commented 1 month ago

I looked at the debug. The INA219 is correctly identified in the intialisation:

_INFO | ??:??:?? 4 INA219 sensor found at address 0x40 INFO | ??:??:?? 4 1 I2C devices found DEBUG | ??:??:?? 4 acc_info = 0 DEBUG | ??:??:?? 4 found i2c sensor meshtastic_TelemetrySensorTypeINA219

I can also see the data being logged as sent: _INFO | ??:??:?? 18 [EnvironmentTelemetryModule] Environment Telemetry: Initializing INFO | ??:??:?? 18 [EnvironmentTelemetryModule] Init sensor: INA219 INFO | ??:??:?? 18 [EnvironmentTelemetryModule] Opened INA219 sensor on i2c bus INFO | ??:??:?? 18 [PowerTelemetryModule] Power Telemetry: Initializing INFO | ??:??:?? 19 [EnvironmentTelemetryModule] (Sending): barometric_pressure=0.000000, current=169.699997, gas_resistance=0.000000, relativehumidity=0.000000, temperature=0.000000 INFO | ??:??:?? 19 [EnvironmentTelemetryModule] (Sending): voltage=0.000000, IAQ=0, distance=0.000000, lux=0.000000 INFO | ??:??:?? 19 [EnvironmentTelemetryModule] (Sending): wind speed=0.000000m/s, direction=0 degrees, weight=0.000000kg DEBUG | ??:??:?? 19 [EnvironmentTelemetryModule] Initial packet id 87601972 DEBUG | ??:??:?? 19 [EnvironmentTelemetryModule] Partially randomized packet id 2250073909 INFO | ??:??:?? 19 [EnvironmentTelemetryModule] Sending packet to mesh

I only see one current value in the telemetry? This resulted in the following being received in the app (now version 2.5.0) Screenshot_20241010_190451_Meshtastic

brebtatv commented 1 month ago

I have a similar problem. In addition, my INA219 has the option of addresses just like a colleague. This makes it impossible to use three INA219s. According to the documentation, other addresses are different.

b8b8 commented 1 month ago

I have a similar problem. In addition, my INA219 has the option of addresses just like a colleague. This makes it impossible to use three INA219s. According to the documentation, other addresses are different.

Use an INA3221 if you need 3 power readings. I have one monitoring load, solar in, and battery. It works great.

brebtatv commented 1 month ago

Screenshot_20241022_173510_Meshtastic There are more of these problems. Current and voltage in the environment? At the same time in Power.

joneyes commented 1 month ago

Also, current reading is displayed in the incorrect units. These values are actually mA.

Screenshot_20241022_124616_Meshtastic.jpg

b8b8 commented 1 month ago

Yup, this stuff was mention already in discord. Remember this is alpha release. Amps to mA should be fixed next release. This is the first version of the app with the new cards, so stuff still to work out. Screenshot_20241023-061424

andrekir commented 1 month ago

all power metrics Telemetry data now moved to the node details screen