Skip to content

Deye/Sunsynk 3-phae inverter sensors for inverter and load added#380

Merged
kellerza merged 3 commits intokellerza:mainfrom
proggaras:deye-3phae-register
Jan 5, 2025
Merged

Deye/Sunsynk 3-phae inverter sensors for inverter and load added#380
kellerza merged 3 commits intokellerza:mainfrom
proggaras:deye-3phae-register

Conversation

@proggaras
Copy link
Copy Markdown
Contributor

Some sensors for the Deye/Sunsynk 3-phae inverter where corrected and added.

  • Inverter L1-L3 voltage
  • Load frequency

Thx. for reviewing.

@maslyankov
Copy link
Copy Markdown
Contributor

    Sensor(630, "Inverter L1 current", AMPS, 0.01),
    Sensor(631, "Inverter L2 current", AMPS, 0.01),
    Sensor(632, "Inverter L3 current", AMPS, 0.01),

According to modbus docs, the above need to be signed like so:

    Sensor(630, "Inverter L1 current", AMPS, -0.01),
    Sensor(631, "Inverter L2 current", AMPS, -0.01),
    Sensor(632, "Inverter L3 current", AMPS, -0.01),

@proggaras
Copy link
Copy Markdown
Contributor Author

@maslyankov
Copy link
Copy Markdown
Contributor

Actually I used this doc: https://forum.iobroker.net/assets/uploads/files/1712644425833-c3ca14a0-178c-4f68-be8c-fa68a414b97b-deye-sun-inverter-modbus-en.pdf

How do you decide if there is a "-" or not?

In the third column you can see it says S16 - signed 16 bits. For signed registers, the code handles them as such when factor is below 0.

@proggaras
Copy link
Copy Markdown
Contributor Author

That makes 100% sense for current. I already pushed a new commit.

@kellerza kellerza merged commit f044dd2 into kellerza:main Jan 5, 2025
@kellerza
Copy link
Copy Markdown
Owner

kellerza commented Jan 5, 2025

Thanks @proggaras!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants