JBD_BLE battery temperature shown as MOSFET temperature in GUI #971
Labels
enhancement
New feature or request
Comments
|
Since there are a few BMS that have a different temperature sensor order maybe it would be an idea that there is a default sensor association which you can overwrite in the config file. |
|
Can you please share a screenshot of the |
|
I'm planing to go to my boat tomorrow. I'll capture your requested info. Just as an advance. My jbd bms only has one ntc. And it's only connected to cells. Bms is protected via 2 thermo switch. All bms apps overkill, basengreen, xiaoxiang detects ntc as cell temp. Mos temp, due to switch, is detected as 0°C. Tomorrow will send info |
|
Here is my \System Logscom.victronenergy.system
Ac/ActiveIn/Frequency 50.1026
Ac/ActiveIn/L1/Current 9.55Ac/ActiveIn/L1/Power 1846
Ac/ActiveIn/L1/Voltage 240.25
Ac/ActiveIn/L2/Current -Ac/ActiveIn/L2/Power -
Ac/ActiveIn/L2/Voltage -
Ac/ActiveIn/L3/Current -Ac/ActiveIn/L3/Power -
Ac/ActiveIn/L3/Voltage -
Ac/ActiveIn/NumberOfPhases 1Ac/ActiveIn/Source 3
Ac/Consumption/Frequency 49.9488
Ac/Consumption/L1/Current 8.91Ac/Consumption/L1/Power 1720
Ac/Consumption/L1/Voltage 240.25
Ac/Consumption/L2/Current -Ac/Consumption/L2/Power -
Ac/Consumption/L2/Voltage -
Ac/Consumption/L3/Current -Ac/Consumption/L3/Power -
Ac/Consumption/L3/Voltage -
Ac/Consumption/NumberOfPhases 1Ac/ConsumptionOnInput/Frequency 50.1026
Ac/ConsumptionOnInput/L1/Current 0
Ac/ConsumptionOnInput/L1/Power 0Ac/ConsumptionOnInput/L1/Voltage 240.25
Ac/ConsumptionOnInput/L2/Current -
Ac/ConsumptionOnInput/L2/Power -Ac/ConsumptionOnInput/L2/Voltage -
Ac/ConsumptionOnInput/L3/Current -
Ac/ConsumptionOnInput/L3/Power -Ac/ConsumptionOnInput/L3/Voltage -
Ac/ConsumptionOnInput/NumberOfPhases 1
Ac/ConsumptionOnOutput/Frequency 49.9488Ac/ConsumptionOnOutput/L1/Current 8.91
Ac/ConsumptionOnOutput/L1/Power 1720
Ac/ConsumptionOnOutput/L1/Voltage 240.25Ac/ConsumptionOnOutput/L2/Current -
Ac/ConsumptionOnOutput/L2/Power -
Ac/ConsumptionOnOutput/L2/Voltage -
Ac/ConsumptionOnOutput/L3/Current
-
Ac/ConsumptionOnOutput/L3/Power -
Ac/ConsumptionOnOutput/L3/Voltage -
Ac/ConsumptionOnOutput/NumberOfPhases 1
Ac/Genset/DeviceType -
Ac/Genset/Frequency -
Ac/Genset/L1/Current -
Ac/Genset/L1/Power -
Ac/Genset/L1/Voltage -
Ac/Genset/L2/Current -
Ac/Genset/L2/Power -
Ac/Genset/L2/Voltage -
Ac/Genset/L3/Current -
Ac/Genset/L3/Power -
Ac/Genset/L3/Voltage -
Ac/Genset/LastStartTime -
Ac/Genset/NumberOfPhases -
Ac/Genset/ProductId -
Ac/Genset/RunningByConditionCode -
Ac/Genset/Runtime -
Ac/Grid/DeviceType -
Ac/Grid/Frequency 50.1026
Ac/Grid/L1/Current 9.55
Ac/Grid/L1/Power 1876
Ac/Grid/L1/Voltage 240.25
Ac/Grid/L2/Current -
Ac/Grid/L2/Power -
Ac/Grid/L2/Voltage -
Ac/Grid/L3/Current -
Ac/Grid/L3/Power -
Ac/Grid/L3/Voltage -
Ac/Grid/NumberOfPhases 1
Ac/Grid/ProductId 9780
Ac/In/0/Connected 0
Ac/In/0/DeviceInstance 276
Ac/In/0/ServiceName com.victronenergy.vebus.ttyS4
Ac/In/0/ServiceType vebus
Ac/In/0/Source 2
Ac/In/0/VrmDeviceInstance 276
Ac/In/1/Connected 1
Ac/In/1/DeviceInstance 276 Ac/In/NumberOfAcInputs 2
Ac/PvOnGenset/L1/Current -
Ac/PvOnGenset/L1/Power -
Ac/PvOnGenset/L2/Current -
Ac/PvOnGenset/L2/Power -
Ac/PvOnGenset/L3/Current -
Ac/PvOnGenset/L3/Power -
Ac/PvOnGenset/NumberOfPhases -
Ac/PvOnGrid/L1/Current -
Ac/PvOnGrid/L1/Power -
Ac/PvOnGrid/L2/Current -
Ac/PvOnGrid/L2/Power -
Ac/PvOnGrid/L3/Current -
Ac/PvOnGrid/L3/Power -
Ac/PvOnGrid/NumberOfPhases -
Ac/PvOnOutput/L1/Current -
Ac/PvOnOutput/L1/Power -
Ac/PvOnOutput/L2/Current -
Ac/PvOnOutput/L2/Power -
Ac/PvOnOutput/L3/Current -
Ac/PvOnOutput/L3/Power -
Ac/PvOnOutput/NumberOfPhases -
ActiveBatteryService com.victronenergy.battery/3
ActiveBmsService com.victronenergy.battery.blea5c2370595b6
AutoSelectedBatteryMeasurement com_victronenergy_battery_3/Dc/0
AutoSelectedBatteryService -
AutoSelectedTemperatureService -
AvailableBatteries {"com.victronenergy.battery/278": {"name": "Shunt DC", "channel": null, "type": "battery"}, "com.victronenergy.ba
AvailableBatteryMeasurements {'com_victronenergy_battery_1024/Dc/0': Battery Aggregator on dbus,'com_victronenergy_battery_278/Dc/0'
AvailableBatteryServices {"default": "Automatic", "nobattery": "No battery monitor", "com.victronenergy.vebus/276": "Quattro 12/3000
AvailableBmsServices [{'instance': 1024,'name': Baterias de Servicio},{'instance': 3,'name': Bateria Serv 2}]
AvailableTemperatureServices {'com.victronenergy.vebus/276/Dc/0/Temperature': Quattro 12/3000/120-2x50 on VE.Bus,'default': Automati
Batteries [{'active_battery_service': 0,'id': com.victronenergy.battery.ttyS6:1,'name': Bateria Arraque,'voltage': 14.41},{'active_b
Buzzer/State 0
Connected 1
Control/ActiveSocLimit -
Control/BatteryCurrentSense 0
Control/BatteryVoltageSense 0
Control/BmsParameters 1
Control/Dvcc 1
Control/EffectiveChargeVoltage 14.4
Control/ScheduledSoc -
Control/SolarChargeCurrent 1
Control/SolarChargeVoltage 1
Control/SolarChargerTemperatureSense 0
Control/SolarChargerVoltageSense 0
Control/VebusSoc 1
Dc/Alternator/Power -
Dc/Battery/Alarms/CircuitBreakerTripped -
Dc/Battery/BatteryService com.victronenergy.battery.blea5c2370595b6
Dc/Battery/ConsumedAmphours 199.2
Dc/Battery/Current 13
Dc/Battery/Power 173.81
Dc/Battery/ProductId 0
Dc/Battery/Soc 70.87
Dc/Battery/State 1
Dc/Battery/Temperature -
Dc/Battery/TemperatureService -
Dc/Battery/TimeToGo 86813
Dc/Battery/Voltage 13.37
Dc/Battery/VoltageService com.victronenergy.battery.blea5c2370595b6
Dc/Charger/Power -
Dc/FuelCell/Power -
Dc/MotorDrive/Power -
Dc/Pv/Current 2
Dc/Pv/Power 26.9
Dc/System/MeasurementType 0
Dc/System/Power 84.3527
Dc/Vebus/Current 18
Dc/Vebus/Power 241
Dc/WindGenerator/Power -
Debug/BatteryOperationalLimits/CurrentOffset 0
Debug/BatteryOperationalLimits/SolarVoltageOffset 0
Debug/BatteryOperationalLimits/VebusVoltageOffset 0
DeviceInstance 0
Dvcc/Alarms/FirmwareInsufficient 0
Dvcc/Alarms/MultipleBatteries 1
DynamicEss/Active 0
DynamicEss/Capabilities 1
DynamicEss/ErrorCode 0
DynamicEss/LastScheduledEnd -
DynamicEss/LastScheduledStart -
DynamicEss/TargetSoc -
DynamicEss/LastScheduledStart -
DynamicEss/TargetSoc -
FirmwareBuild 20240124050751
FirmwareVersion 201728
GpsService -
HardwareVersion -
Hub 1
Mgmt/Connection data from other dbus processes
Mgmt/ProcessName /opt/victronenergy/dbus-systemcalc-py/dbus_systemcalc.py
Mgmt/ProcessVersion 2.140
ProductId -
ProductName -
PvInvertersProductIds -
Relay/0/State 0
Relay/1/State 0
Serial d4124390d6ca
ServiceMapping/com_victronenergy_battery_1024 com.victronenergy.battery.aggregator
ServiceMapping/com_victronenergy_battery_278 com.victronenergy.battery.ttyS6
ServiceMapping/com_victronenergy_battery_3 com.victronenergy.battery.blea5c2370595b6
ServiceMapping/com_victronenergy_settings_0 com.victronenergy.settings
ServiceMapping/com_victronenergy_solarcharger_277 com.victronenergy.solarcharger.ttyS5
ServiceMapping/com_victronenergy_solarcharger_279 com.victronenergy.solarcharger.ttyS7
ServiceMapping/com_victronenergy_temperature_20 com.victronenergy.temperature.adc_builtin0_8
ServiceMapping/com_victronenergy_temperature_21 com.victronenergy.temperature.adc_builtin0_7
ServiceMapping/com_victronenergy_temperature_22 com.victronenergy.temperature.adc_builtin0_6
ServiceMapping/com_victronenergy_temperature_23 com.victronenergy.temperature.adc_builtin0_5
ServiceMapping/com_victronenergy_vebus_276 com.victronenergy.vebus.ttyS4
ServiceMapping/com_victronenergy_vecan_0 com.victronenergy.vecan.can0
SystemState/BatteryLife 0
SystemState/ChargeDisabled 0
SystemState/DischargeDisabled 0
SystemState/LowSoc 0
SystemState/SlowCharge 0
SystemState/State 252
SystemState/UserChargeLimited 0
SystemState/UserDischargeLimited 0
SystemType Hub-1
Timers/TimeOff 147050
Timers/TimeOnGenerator 0
Timers/TimeOnGrid 8006
Timers/TimeOnInverter 2724
VebusInstance 276 |
|
@mr-manuel jus to let you know that my system config is published as requested... |
|
Unfortunately we misunderstood each other. I need the
|
|
OK... sorry... @mr-manuel here they are Logscom.victronenergy.battery.ble_a5c2370595b6 com.victronenergy.battery.ble_a5c2370595b6 com.victronenergy.battery.ble_a5c2370595b6 com.victronenergy.battery.ble_a5c2370595b6 |
mr-manuel
added a commit
to mr-manuel/venus-os_dbus-serialbattery
that referenced
this issue
Feb 27, 2024
|
Could you test wget -O /tmp/install.sh https://raw.githubusercontent.com/Louisvdw/dbus-serialbattery/master/etc/dbus-serialbattery/install.sh
bash /tmp/install.shIf there is only one temperature sensor it is used as battery sensor. |
|
Nice idea. Single temp sensor always on battery cells. ==> /data/log/dbus-blebattery.0/current <== |
|
On the first restart another error came. 2024-02-27 11:22:57.224888500 INFO:SerialBattery: |
|
Can you contact me on Discord? Since I don't know which data it outputs exactly I would save much time, when I can connect directly to your system. |
mr-manuel
added a commit
to mr-manuel/venus-os_dbus-serialbattery
that referenced
this issue
Feb 27, 2024
|
The problem was fixed with |
Louisvdw
pushed a commit
that referenced
this issue
Feb 28, 2024
* fix Sinowealth not loading #702 * fix unique identifier function * enable BMS over config, if disabled by default Now you can also add more then one BMS for BMS_TYPE * show battery port in log * ANT BMS fixes Fixed that other devices are recognized as ANT BMS * Sinowealth BMS fixes Fixed that other devices are recognized as Sinowealth BMS * improved publish_battery error handling switched from error count to seconds * Improve Battery Voltage Handling in Linear Absorption Mode * Refactor change time() to int(time()) for consistency in max_voltage_start_time and tDiff calculation * Refactor battery voltage calculations for efficiency and clarity * Remove penalty_buffer * Reset max_voltage_start_time wenn we going to bulk(dynamic) mode * updated changelog * fix reply processing * Reduce the big inrush current, if the CVL jumps from Bulk/Absorbtion to Float fix #659 * Check returned data lenght for Seplos BMS Be stricter about the return data we accept, might fix the problem of grid meters accidently being recognized as a Seplos * Validate current, voltage, capacity and SoC for all BMS This prevents that a device, which is no BMS, is detected as BMS * removed double check * bump version * fix validation if None * updated changelog * proposal to #659 formatted :) * bugfix proposal to #659 * refactor setting float charge_mode * fix type error, removed bluetooth cronjob * updated changelog * fix rs485 write communication errors by inserting sleeps, add debug print for charge mode and fix crash on write soc failures * fix write problem on set_soc. also changed the switch charge/discharge function, just in case * debug msg * Bluetooth optimizations * Fixes by @peterohman #505 (comment) * fix #712 * fix meaningless time to go values * fix meaningless time to go values * Duration of transition to float depends on number of cells * Float transition - Voltage drop per second * Update hlpdatabms4s.py * Validate setting of FLOAT_CELL_VOLTAGE and avoid misconfiguration * consider utils.LINEAR_RECALCULATION_EVERY to refresh CVL * cleanup * consider utils.LINEAR_RECALCULATION_EVERY to refresh CVL * small refactor, introduced set_cvl_linear function to set CVL only once every LINEAR_RECALCULATION_EVERY seconds * fix typo * updated changelog * remove debug msg * remove debug msg * undo debug change * Daly BMS make auto reset soc configurable * added debug and error information for CVL * fix proposal for #733 (#735) * Added: Tollerance to enter float voltage once the timer is triggered * Add bulk voltage Load to bulk voltage every x days to reset the SoC to 100% for some BMS * JKBMS disable high voltage warning on bulk reenable after bulk was completed * fixed error * disable high voltage warning for all BMS when charging to bulk voltage * fix error and change default value measurementToleranceVariation from 0.025 to 0.5 else in OffGrid mode max voltage is always kept * Added temperature names to dbus/mqtt * Use current avg of last 300 cycles for TTG & TTS * Calculate only positive Time-to-SoC points * added current average of last 5 minutes * make CCL and DCL more clear * fix small error * bugfix: LLTJBD BMS SOC different in Xiaoxiang app and dbus-serialbattery * black formatting * JDB BMS - Control FETs for charge, discharge and disable / enable balancer (#761) * feature: Allow to control charge / discharge FET * feature: Allow to enable / disable balancer * bugfix: Cycle Capacity is in 10 mAh Fixes SoC with factor 100 * 100% percentage * JBD BMS show balancer state in GUI page IO (#763) * Bump version * Fix typos * Smaller fixes - fixes #792 (comment) * Removed comments from utils.py This should make more clear that there are no values to change * Updated changelog * possible fix for LLT/JBS connection problems #769 #777 * bugfix: LLT/JBD BMS general packet data size check * improved reinstall and disable script * LLT/JBD BMS - Improved error handling and automatical driver restart in case of error. Should fix: - #730 - #769 - #777 * Fixed Building wheel for dbus-fast won't finish on weak systems Fixes #785 * Support for Daly CAN Bus (#169) * support for Daly CAN Bus * fix constructor args * revert port, needs fix * add can filters * comment logger Some changes are still needed to work with the latest version. They will follow in a next PR. --------- Co-authored-by: Samuel Brucksch <[email protected]> Co-authored-by: Manuel <[email protected]> * JKBMS BLE - Introduction of automatic SOC reset (HW Version 11) (#736) * Introduction of automatic SOC reset for JK BMS (HW Version 11) * Fixed value mapping * Rework of the code to make it simpler to use without additional configuration. Moved execution of SOC reset. It's now executed while changing from "Float" to "Float Transition". * Implementation of suggested changes Persist initial BMS OVP and OVPR settings Make use of max_cell_voltage to calculate trigger value for OVP alert * Added: Daly CAN and JKBMS CAN * added CAN bms to installation script optimized CAN drivers * smaller fixes * Trigger JK BLE SOC reset when using Step Mode * Moved trigger_soc_reset() * fixes LLT/JBD SOC > 100% #769 * changed VOLTAGE_DROP behaviour * Fix JKBMS not starting if BMS manuf. date is empty * corrected bulk, absorption and soc reset terms * fix typo * add JKBMS_BLE debugging data * fix small error * Some changes for lost bluetooth connection / hci_uart stack restart * added logging to config * add sleep before starting driver prevents lot of timeouts after reinstalling the driver, since the restart is now much faster than before * changed post install info * fix error * Daly BMS fixed embedded null byte #837 * added info for SoC reset to default config file * fix for #716 #716 * fix for #716 and JKBMS model recognition #716 * optimized logging * fix JKBMS recognition * added debugging * fixes #716 #716 * Bind device instance to unique_identifier #718 * added data types to battery class disabled unused variables * save current charge state #840 * correct file permissions * updated changelog * added periodic saveChargeDetails * fix some small errors * fix issue with ruuvi tags When there are hundreds of unused ruuvi tags in the settings list that where added because thei where nearby the driver does not start correctly. These stale entries are disabled on the driver startup. The issue was already filed to Victron developers * CVL with i-controller instead of penaltysum * cvl_controller: switch to choose PenaltySum or ICOntroller + documentation * docu enhancement * Add setting and install logic for usb bluetooth module * round temperatures * changed battery disconnect behaviour * Fixes #891 #891 * updated changelog * Add bluetooth device note to config.default.ini * Fix typo in bluetooth note in config.default.ini * fixed error in new cvl_controller * fixed float division by zero and code optimization * Restart MAX_VOLTAGE_TIME_SEC if cell diff > CELL_VOLTAGE_DIFF_KEEP_MAX_VOLTAGE_TIME_RESTART * Calculation of the SOC based on coloumb-counting (#868) * Calculation of the SOC in the driver based on coloumb-counting * soc_calc: add current correction before integration * soc_calc: correction map for current * Soc_calc: CorrectionMap, switch to turn on/off correction, selectable initial value * soc_calc: Bugfix * soc_calc: Bugfix * store soc in dbus for restart * store soc in dbus for restart (formatted) * store soc in dbus for restart (bugfix) * save soc_calc only after change > 1.0 * store soc in dbus for restart (bugfix) * logger does not work this way. do not know why * writing and reading to dbus works * Removed options: SOC_CALC_CURRENT_CORRECTION, SOC_CALC_RESET_VALUE_ON_RESTART, SOC_CALC_INIT_VALUE sort soc_calc alphabetically * fixed comments * Updated changelog, small fixes * Changed: PUBLISH_CONFIG_VALUES from 0/1 to True/False * Changed: Code optimizations - Changed some variables to be more clear - Added comments for easier code understanding * Calculated SOC: Added two decimals, added BMS SOC for MQTT & Node-RED * Updated changelog, small fixes * Changed: PUBLISH_CONFIG_VALUES from 0/1 to True/False * Changed: Code optimizations - Changed some variables to be more clear - Added comments for easier code understanding * Calculated SOC: Added two decimals, added BMS SOC for MQTT & Node-RED * Fix #898 #898 * Changed: Fix issue loading settings from dbus * Added nightly install option makes it easier for users to pretest fixes * Changed: more detailed error output when an exception happens * Possible fix for #912 #912 * Fixes #919 #919 * Changed: Exit script with error, if port excluded else the serialstarter stops at the dbus-serialbattery * Fixed some smaller errors * Updated pre-release workflow * Fix JK BMS connection restart when bluetooth fails. This fix installs a new thread to monitor the state of the original scraping thread. If scraping thread dies, it verifies that it did not because the scraping was intentionally stopped by calling stop_scrapping. When restarting the scrapper, it first calls the bluetooth reset lambda function that was passed in the class contructor, such that bluetooth is ready to make a proper connection. * Fixes #916 #916 * Added Venus OS version to logfile * Fix #840 #840 * Small code formatting fixes * Optimized reinstall script. Restart GUI only on changes. * Display debugging data in GUI when DEBUG enabled * Install script now shows repositories and version numbers * Update daly_can.py Fixing #950 for DalyBMS * Update jkbms_can.py Fixing #950 for Jk BMS * Fix black lint check * Fixes #970 #970 * Fixed some errors in restoring values from dbus settings * Moved sleep on start for all BMS * Update config description * Reworked a part of the default config * fix typo in stopping services when reinstalling * Fix Time-to-SoC and Time-to-Go calculation * Add changelog info * Round sum and diff voltage * Temperature limitation variables where changed * SoC limitation variables where changed * Added error messages * Remove unneeded code * Reset SoC to 0% if empty * Add GUIv2 for dbus-serialbattery * Check free space before installing * Added new GUIv2 version * Removed Python 2 compatibility * Changelog update * Code cleanup - Removed: get_temperatures() - Removed: update_last_seen() * Bluetooth code optimizations * Fixed some JKBMS BLE not starting #819 * Check if packages are already installed before install * Fixed some SOC calculation errors * Fixed None SOC on driver start * Do not show and allow button change when callback is missing for: - ForceChargingOff - ForceDischargingOff - TurnBalancingOff * Check if a device instance is already used by creating a PID file * Log and execute SOC reset to 100% or 0% only once * Update GitHub workflow and issue templates * Fixed LLT/JBD BMS with only on temperature sensor #791 #971 * Fix warning on reinstall * Fix missing IO control for JBDBMS #992 #992 * Prepare for removing dev branch --------- Co-authored-by: ogurevich <[email protected]> Co-authored-by: Bernd Stahlbock <[email protected]> Co-authored-by: wollew <[email protected]> Co-authored-by: Oleg Gurevich <[email protected]> Co-authored-by: peterohman <[email protected]> Co-authored-by: Strawder, Paul <[email protected]> Co-authored-by: Paul Strawder <[email protected]> Co-authored-by: Samuel Brucksch <[email protected]> Co-authored-by: Samuel Brucksch <[email protected]> Co-authored-by: ArendsM <[email protected]> Co-authored-by: Meik Arends <[email protected]> Co-authored-by: Marvo2011 <[email protected]> Co-authored-by: cflenker <[email protected]> Co-authored-by: cflenker <[email protected]> Co-authored-by: Cupertino Miranda <[email protected]> Co-authored-by: Martin Polehla <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
I Can't use new temperature protections functionality due to wrong BMS temperature assignments. Seems that driver detects first temperature sensor as MOSFET temperature but actually many batteries in the market instead of MOSFET NTC, they have just a termoswitch that triggers upon certain temperature. So in those BMSs, the temperature sensor is attached to the cells and not the MOS. My BMS is for instance, JBD DP04S007. Quite common model.
Describe the solution you'd like
Option to select if NTC 1 Correspond to a Cell or MOSFET
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: