diff --git a/README.MD b/README.MD index 12272a5..7476a5e 100644 --- a/README.MD +++ b/README.MD @@ -8,7 +8,8 @@ This project embeds the Chargery BMS to a Victron Energy Venus OS device to moni ![Overview](https://raw.githubusercontent.com/Tobi177/venus-chargerybms/master/examples/overview.png) ![ChargeryBMS](https://raw.githubusercontent.com/Tobi177/venus-chargerybms/master/examples/chargerybms.png) -![ChargeryBMS Cells](https://raw.githubusercontent.com/Tobi177/venus-chargerybms/master/examples/chargerybmscells.png) +![ChargeryBMS Voltages](https://raw.githubusercontent.com/Tobi177/venus-chargerybms/master/examples/chargerybmsvoltages.png) +![ChargeryBMS Impedances](https://raw.githubusercontent.com/Tobi177/venus-chargerybms/master/examples/chargerybmsimpedances.png) # Requirements @@ -21,9 +22,6 @@ This project embeds the Chargery BMS to a Victron Energy Venus OS device to moni - TBD # Notes -- Proof-of-Concept style -- Only the Chargery BMS16T is currently supported. However the sources can very easily be changed to support BMS8T and BMS24T as well. -- No multiple languages - No scripts to resetup the changes automatically on update of Venus OS -- The devices are "hard-coded" at the qml files, must be adapted (VE.direct devices, etc) +- The devices are "hard-coded" at the overview qml file, must be adapted (VE.direct devices, etc) diff --git a/driver/chargerybms.py b/driver/chargerybms.py index 5b7659c..0c5f8fe 100644 --- a/driver/chargerybms.py +++ b/driver/chargerybms.py @@ -881,7 +881,7 @@ def parse_packet(packet): # print (BMS_STATUS) - logging.info("BMS status [SOC|" + BMS_STATUS['bms']['soc']['text'] + + logging.info("BMS Status [SOC|" + BMS_STATUS['bms']['soc']['text'] + "][MODE|" + BMS_STATUS['bms']['current_mode']['text'] + "][CURRENT|" + BMS_STATUS['bms']['current']['text'] + "][T1|" + BMS_STATUS['bms']['temperature']['sensor_t1']['text'] + @@ -1242,7 +1242,7 @@ def parse_packet(packet): # print (BMS_STATUS) if (packet_length == PACKET_LENGTH_STATUS_CELLS[0]): # packet from BMS8 - logging.info("BMS8 cells " + + logging.info("BMS Voltages " + "[CAPACITYAH|" + BMS_STATUS['voltages']['battery_capacity_ah']['text'] + "][CAPACITYWH|" + BMS_STATUS['voltages']['battery_capacity_wh']['text'] + "][DIFF|" + BMS_STATUS['voltages']['agg_voltages']['diff']['text'] + @@ -1258,7 +1258,7 @@ def parse_packet(packet): elif (packet_length == PACKET_LENGTH_STATUS_CELLS[1]): # packet from BMS16 - logging.info("BMS16 cells " + + logging.info("BMS Voltages " + "[CAPACITYAH|" + BMS_STATUS['voltages']['battery_capacity_ah']['text'] + "][CAPACITYWH|" + BMS_STATUS['voltages']['battery_capacity_wh']['text'] + "][DIFF|" + BMS_STATUS['voltages']['agg_voltages']['diff']['text'] + @@ -1283,7 +1283,7 @@ def parse_packet(packet): elif (packet_length == PACKET_LENGTH_STATUS_CELLS[2]): # packet from BMS24 - logging.info("BMS24 cells " + + logging.info("BMS Voltages " + "[CAPACITYAH|" + BMS_STATUS['voltages']['battery_capacity_ah']['text'] + "][CAPACITYWH|" + BMS_STATUS['voltages']['battery_capacity_wh']['text'] + "][DIFF|" + BMS_STATUS['voltages']['agg_voltages']['diff']['text'] + @@ -1459,7 +1459,7 @@ def parse_packet(packet): dbusservice["/Impedances/UpdateTimestamp"] = BMS_STATUS['impedances']['timestamp']['text'] dbusservice["/Raw/Impedances/UpdateTimestamp"] = BMS_STATUS['impedances']['timestamp']['value'] - logging.info("BMS impedances " + + logging.info("BMS Impedances " + "][MODE1|" + BMS_STATUS['impedances']['current_mode1']['text'] + "][CURRENT1|" + BMS_STATUS['impedances']['current1']['text'] + "][SUM|" + BMS_STATUS['impedances']['agg_impedances']['sum']['text'] + diff --git a/examples/chargerybms.png b/examples/chargerybms.png index 4f7bb63..febe464 100644 Binary files a/examples/chargerybms.png and b/examples/chargerybms.png differ diff --git a/examples/chargerybmscells.png b/examples/chargerybmscells.png deleted file mode 100644 index fa9ffe3..0000000 Binary files a/examples/chargerybmscells.png and /dev/null differ diff --git a/examples/chargerybmsimpedances.png b/examples/chargerybmsimpedances.png new file mode 100644 index 0000000..ec856cd Binary files /dev/null and b/examples/chargerybmsimpedances.png differ diff --git a/examples/chargerybmsvoltages.png b/examples/chargerybmsvoltages.png new file mode 100644 index 0000000..576ae9e Binary files /dev/null and b/examples/chargerybmsvoltages.png differ diff --git a/examples/overview.png b/examples/overview.png index 5e75089..3af7270 100644 Binary files a/examples/overview.png and b/examples/overview.png differ diff --git a/gui/qml/PageBatteryChargeryBMS.qml b/gui/qml/PageBatteryChargeryBMS.qml index 7216930..e4c18b3 100644 --- a/gui/qml/PageBatteryChargeryBMS.qml +++ b/gui/qml/PageBatteryChargeryBMS.qml @@ -75,7 +75,7 @@ MbPage { MbSubMenu { - description: qsTr("Cells Voltages") + description: qsTr("Cell Voltages") subpage: Component { PageBatteryChargeryBMSVoltages { bindPrefix: service.path("") @@ -84,7 +84,7 @@ MbPage { } MbSubMenu { - description: qsTr("Cells Impedances") + description: qsTr("Cell Impedances") subpage: Component { PageBatteryChargeryBMSImpedances { bindPrefix: service.path("") diff --git a/gui/qml/PageBatteryChargeryBMSImpedances.qml b/gui/qml/PageBatteryChargeryBMSImpedances.qml index 01d3302..197a1fb 100644 --- a/gui/qml/PageBatteryChargeryBMSImpedances.qml +++ b/gui/qml/PageBatteryChargeryBMSImpedances.qml @@ -7,7 +7,7 @@ MbPage { property string bindPrefix property BatteryDetails details: BatteryDetails { bindPrefix: root.bindPrefix } - title: service.description + " | Cells Impedances" + title: service.description + " | Cell Impedances" model: VisualItemModel { diff --git a/gui/qml/PageBatteryChargeryBMSVoltages.qml b/gui/qml/PageBatteryChargeryBMSVoltages.qml index f55cf1f..d566f3f 100644 --- a/gui/qml/PageBatteryChargeryBMSVoltages.qml +++ b/gui/qml/PageBatteryChargeryBMSVoltages.qml @@ -7,7 +7,7 @@ MbPage { property string bindPrefix property BatteryDetails details: BatteryDetails { bindPrefix: root.bindPrefix } - title: service.description + " | Cells Voltages" + title: service.description + " | Cell Voltages" model: VisualItemModel {