Skip to content

Conversation

@paultweedy77
Copy link
Contributor

Post Manager v1.44 (and confirmed in later v1.49), EPS Combined Power is present on 39216/39217 (I32) for H1_G2 inverters, as per latest modbus spec as quoted in #883 (reply in thread)

@paultweedy77
Copy link
Contributor Author

Hm, I'm making a right mess of fixing the tests and don't have enough Python skills to verify all of this locally, so I'm going to close this PR.

@MartB
Copy link
Contributor

MartB commented Oct 29, 2025

@paultweedy77 Would you mind picking this up again? Is there anything i can help you with?
If you set-up the recommended dev-container in vscode you can just run
pytest --snapshot-update to fix the tests.

@paultweedy77
Copy link
Contributor Author

paultweedy77 commented Nov 1, 2025

@paultweedy77 Would you mind picking this up again? Is there anything i can help you with? If you set-up the recommended dev-container in vscode you can just run pytest --snapshot-update to fix the tests.

Unfortunately I don't have the time or patience to set up and use vscode .. I'm an occasional emacs/vi sort of code basher these days, so happy adding the different modbus registers in the entity descriptions file (which works well) but setting up CI is beyond me. Would be great if someone else could do this :)

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.

2 participants