-
-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Commit #393: BMS-own blocking charge/discharge not available #458
Comments
Hi @Dr-Gigavolt, I try to understand your problem and search for a possible solution. Unfortunately I do not fully understand the problem.
What else should write on this paths, if not the driver?
What do you mean with this?
Could you check, if the problem persists when you install the code from the To install execute this wget -O /tmp/install-nightly.sh https://raw.githubusercontent.com/Louisvdw/dbus-serialbattery/jkbms_ble/etc/dbus-serialbattery/install-nightly.sh && bash /tmp/install-nightly.sh and select |
Hi @mr-manuel |
Sorry, but I still can't understand. The code from the
|
self._dbusservice["/System/NrOfModulesBlockingCharge"] = ( | |
0 | |
if self.battery.charge_fet is None | |
or (self.battery.charge_fet and self.battery.control_allow_charge) | |
else 1 | |
) | |
self._dbusservice["/System/NrOfModulesBlockingDischarge"] = ( | |
0 if self.battery.discharge_fet is None or self.battery.discharge_fet else 1 | |
) |
v0.14.3
dbus-serialbattery/etc/dbus-serialbattery/dbushelper.py
Lines 244 to 247 in 3685d15
self._dbusservice['/System/NrOfModulesBlockingCharge'] = 0 if self.battery.charge_fet is None or \ | |
(self.battery.charge_fet and self.battery.control_allow_charge) else 1 | |
self._dbusservice['/System/NrOfModulesBlockingDischarge'] = 0 if self.battery.discharge_fet is None \ | |
or self.battery.discharge_fet else 1 |
v0.14.0
dbus-serialbattery/etc/dbus-serialbattery/dbushelper.py
Lines 244 to 247 in 7aab4c8
self._dbusservice['/System/NrOfModulesBlockingCharge'] = 0 if self.battery.charge_fet is None or \ | |
(self.battery.charge_fet and self.battery.control_allow_charge) else 1 | |
self._dbusservice['/System/NrOfModulesBlockingDischarge'] = 0 if self.battery.discharge_fet is None \ | |
or self.battery.discharge_fet else 1 |
v0.13.0
dbus-serialbattery/etc/dbus-serialbattery/dbushelper.py
Lines 211 to 214 in 6393e76
self._dbusservice['/System/NrOfModulesBlockingCharge'] = 0 if self.battery.charge_fet is None or \ | |
(self.battery.charge_fet and self.battery.control_allow_charge) else 1 | |
self._dbusservice['/System/NrOfModulesBlockingDischarge'] = 0 if self.battery.discharge_fet is None \ | |
or self.battery.discharge_fet else 1 |
However between v0.13.0
and v0.14.0
it was changed how control_allow_charge
is set: Comparing v0.13...v0.14 · Louisvdw/dbus-serialbattery. But this was one month before the commit you are referring to.
Am I missing something?
There are new changes regarding this values: @Dr-Gigavolt any update from your side? |
Hi @mr-manuel, I am sorry, I had no time yet. |
Hi @mr-manuel , I've just installed the jkbms_ble and will observe the behavior tomorrow. I appreciate the well commented config.default.ini. But please explain me why the lead-acid algorithm bulk-absorption-float is still maintained? This art of choreography was developed for lead-acid cells having no balancer, no BMS and high leakage current. |
There are only two voltages:
|
Hi @mr-manuel,
|
Which BMS do you have? Maybe Daly? If yes we imrpoved the driver a lot and now it runs finally smoothly. Would be great, if you can test it. It was released as beta today. If the battery monitor is not available, then it will fall back to the default for a MultiPlus. There is no way to block discharge/charge, but you can prevent bigger currents by setting this settings to GUI -> Settings -> DVCC -> Limit charge current |
@Dr-Gigavolt can you check /var/log/messages.* for "watchdog" and "load average too high"? I've seen that before and wonder if it is the same |
No, I have JK. |
I installed the jkbms-ble again to see the messages after the next crash. |
The first crash already happened: |
Could you post a screenshot of the result from the top command? Please watch for a minute how it's ongoing, so you get a feeling, when the best moment for a screenshot is. Else it can happen that you take a screenshot where the system is idle for a second. I had a similar problem, when the device run out of memory. Then it bebun to swap and the CPU load went to the sky. This was caused by Node-RED. |
I don't understand what you mean. The crash happens after some hours, I cannot sit and wait for it. But this commit works reliably and I use it since January: https://github.com/Louisvdw/dbus-serialbattery/tree/3685d15a5d1ba801867bbdb98f57fd805ea2c24a/etc/dbus-serialbattery |
Reboot your Venus OS and check after 4 hours for one minute the |
What is the "top command"? |
@Dr-Gigavolt I think this is what is meant "top Command" https://www.linux-praxis.de/top-zeigt-laufende-cpu-prozesse |
Hello @mr-manuel, |
I close this issue, since the issue related problem is solved. @Dr-Gigavolt there is another issue open related to your current problem #655 |
Hi Louis,
since the commit #393 the paths
/System/NrOfModulesBlockingCharge
and/System/NrOfModulesBlockingDischarge
are always set by your code, regardless whetherCCCM_CV_ENABLE
andDCCM_CV_ENABLE
are setTrue
orFalse
. It is not possible to transmit the state of MOSFET switch any more. Is there some reason for it? Otherwise please make the possibility to pass the BMS state again. Thanks.Best regards,
Anton
The text was updated successfully, but these errors were encountered: