dreadnought / python-daly-bms

Python module for Daly BMS devices
MIT License
82 stars 37 forks source link

No cell_voltages or temperatures Data, Except There Is? #42

Closed kneave closed 11 months ago

kneave commented 11 months ago

Howdo,

I've a DALY BMS, it seems to be one of the non-sinowealth ones, and I'm able to get loads of data back over UART so it's working. The odd thing is that though it's telling me how many cells I have, and the high and low values, cell_voltages is null. Same for temperature.

This a known issue or a new one? I couldn't spot it in the open or closed issues so thought I'd raise one.

daly-bms-cli --uart -d /dev/ttyUSB0 --all
ERROR    [daly_bms.py:257] unkonwn status_field cells
ERROR    [daly_bms.py:257] unkonwn status_field temperature_sensors
{
  "soc": {
    "total_voltage": 35.9,
    "current": -1.6,
    "soc_percent": 99.1
  },
  "cell_voltage_range": {
    "highest_voltage": 3.601,
    "highest_cell": 10,
    "lowest_voltage": 3.596,
    "lowest_cell": 1
  },
  "temperature_range": {
    "highest_temperature": 22,
    "highest_sensor": 1,
    "lowest_temperature": 22,
    "lowest_sensor": 1
  },
  "mosfet_status": {
    "mode": "charging",
    "charging_mosfet": true,
    "discharging_mosfet": true,
    "capacity_ah": 12.387
  },
  "status": {
    "cells": 10,
    "temperature_sensors": 1,
    "charger_running": false,
    "load_running": false,
    "states": {
      "DI1": false,
      "DI2": true
    },
    "cycles": 0
  },
  "cell_voltages": null,
  "temperatures": null,
  "balancing_status": {
    "error": "not implemented"
  },
  "errors": []
}
kneave commented 11 months ago

It's now reporting voltages, no idea why but closing as no-repro

{
  "soc": {
    "total_voltage": 41.8,
    "current": -0.3,
    "soc_percent": 99.1
  },
  "cell_voltage_range": {
    "highest_voltage": 4.187,
    "highest_cell": 9,
    "lowest_voltage": 4.181,
    "lowest_cell": 1
  },
  "temperature_range": {
    "highest_temperature": 22,
    "highest_sensor": 1,
    "lowest_temperature": 22,
    "lowest_sensor": 1
  },
  "mosfet_status": {
    "mode": "charging",
    "charging_mosfet": true,
    "discharging_mosfet": true,
    "capacity_ah": 12.387
  },
  "status": {
    "cells": 10,
    "temperature_sensors": 1,
    "charger_running": false,
    "load_running": false,
    "states": {
      "DI1": false,
      "DI2": true
    },
    "cycles": 1
  },
  "cell_voltages": {
    "1": 4.181,
    "2": 4.185,
    "3": 4.185,
    "4": 4.185,
    "5": 4.183,
    "6": 4.185,
    "7": 4.185,
    "8": 4.187,
    "9": 4.187,
    "10": 4.185
  },
  "temperatures": {
    "1": 22
  },
  "balancing_status": {
    "error": "not implemented"
  },
  "errors": [ ]
}