nathanmarlor / foxess_modbus

FoxESS inverter integration. Connect directly to your FoxESS inverter (no cloud!) for real-time status and control.
MIT License
105 stars 14 forks source link

Load sensor artificially low after latest beta release. #571

Closed ryaaanmorgan closed 3 months ago

ryaaanmorgan commented 3 months ago

Evening lads, noticed my "load" sensor hit rock bottom despite all other sensors (including the CT) being correct. I've done some digging and really can't explain it. It's not done wonders for my losses sensor 🤣

Any thoughts? It coincided with the upgrade to the latest beta last night and I think that might be a bug. Nothing in the logs to suggest an issue neither.

image

ryaaanmorgan commented 3 months ago

HA is at version 2024.3.0

ryaaanmorgan commented 3 months ago

I've moved back to the most recent non-beta release and the load sensor is fixed. Any details you need to help recreate the error - let me know.

FozzieUK commented 3 months ago

Morning Ryan, thanks for spotting that :)

@canton7 - looks like an error crept in when refactoring - see https://github.com/nathanmarlor/foxess_modbus/pull/572