Fixes for zcs_azzurro-hyd-zss-hp.yaml #397
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Battery DOD (depth of discharge) - Battery EOD (end of discharge) - Battery EPS buffer - Load Voltage - Load Frequency - Today Load Consumption - Total Load Consumption - Serial Number (inverter) - Hardware Version - Software Version Master - Software Version Slave - Safety Version - Safety Firmware Version - Safety Hardware Version
This value is not documented in Sofar specs, it may be equivalent to "Grid Failure" in Huawei inverters but it doesn't have its own error ID. It's fired when the power grid experiences an outage, the AC power cable is disconnected or the AC circuit breaker is OFF, and "Device State" is "Emergency power supply" It's only fired when the "Load" port has a connection, otherwise you'll only get a "ID02 Grid Under Voltage Protection", so it's more of a "state" then an actual "error".
- 0x0003 should be "Emergency power supply" - 0x0004 should be "Recoverable fault" - 0x0005 should be "Permanent fault" - 0x0006 should be "Upgrade" (state) Please check the reference for register addres 0404 : https://github.com/user-attachments/files/17180102/SOFAR-G3.External.Modbus.Protocol-EN-V1.10.20220622.xlsx
…as soon as possible if your inverter has a problem...
…know as soon as possible if it drops below a threshold...
…oon as possible if your inverter has a problem...
…frequently so there's little value in retrieving them every 5 seconds
|
Hi @gionaf,
I think it should be
Thanks. |
|
Hi @davidrapan ,
yes, sounds right
i see you already merged, are those changes still needed? |
Nono, I solved it by merging just the second PR (which also contained commits from the previous one). 😉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note: this pull request requires my other branch ("giona_zcs_azzurro-hyd-zss-hp_new-sensors") to be merged first!
Device states didn't match Sofar specifications:
Please check the reference for register addres 0404 : https://github.com/user-attachments/files/17180102/SOFAR-G3.External.Modbus.Protocol-EN-V1.10.20220622.xlsx
Battery
Update intervals