Skip to content
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

Daly BMS reporting charging when discharging and Amps have inverted sign #65

Closed
ddarek2000 opened this issue Oct 27, 2021 · 5 comments · Fixed by #66
Closed

Daly BMS reporting charging when discharging and Amps have inverted sign #65

ddarek2000 opened this issue Oct 27, 2021 · 5 comments · Fixed by #66

Comments

@ddarek2000
Copy link
Contributor

ddarek2000 commented Oct 27, 2021

Daly Smart BMS 150A 16S has a firmware problem that reports inverted sign for Amps. This results in BMS reporting charging when inverter is running , and BMS discharing when the charger is running.

I would like to have a variable / switch that would cause the values to invert when such a problem is detected by the user. This should be manual configuration , not an automatic switch.

I am considering replacing the BMS.

BMS is connected using UART to USB connection which works perfectly. Above scenario is also observed using Bluetooth IOS app and PC app
Daly_Smart_BMS
.

@pau1phi11ips
Copy link

This is odd.

I'm using a 150A 8S Daly and the current direction on this driver is the same as the current direction reported by my Victron BMV-712

@21-Jailbraik-21
Copy link

Das gleiche Problem habe ich auch mit dem Daly 16S 120A -

@Louisvdw
Copy link
Owner

I suspect that there might be different firmwares on the BMS where this is swopped. I have another earlier issue #36 with a similar scenario, while the original BMS used when writing the driver show it the correct way,
So it seems like a switch in the code is the correct way to handle this.

@21-Jailbraik-21
Copy link

That would make me happy and very happy.
Thank you for the great job

@Louisvdw
Copy link
Owner

In the next release you can change the utils.py to invert the current measurement
INVERT_CURRENT_MEASUREMENT = -1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants