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

Protocol description for SG is mixed up #584

Open
Celphor opened this issue Nov 17, 2024 · 2 comments
Open

Protocol description for SG is mixed up #584

Celphor opened this issue Nov 17, 2024 · 2 comments

Comments

@Celphor
Copy link

Celphor commented Nov 17, 2024

Descriptions in the decyption document for byte 71 and 72 are swapped, as well as 73 and 74

Correct is:

71 = SG Setting 1 DHW Capacity
72 = SG Setting 1 Heating Capacity
73 = SG Setting 2 DHW Capacity
74 = SG Setting 2 Heating Capacity

I can confirm SG works according to the manual when external SG switches are applied

@geduxas
Copy link
Contributor

geduxas commented Nov 17, 2024

It's only in documentation or in code also?

@Celphor
Copy link
Author

Celphor commented Nov 17, 2024

Only in documentation. There is no TOP for it.

There seem to be no shadow bits für the SG inputs in the protocol. At least i didn't find any changes in the heatpump reply.

Also the SG function is pretty useless for heishamon users. Increasing/decreasing heatcurve or dhw temp shifts can be derived with existing TOPs much better. No need to put any effort on it

@Celphor Celphor changed the title Protocol description for SG is mixup up Protocol description for SG is mixed up Nov 17, 2024
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

No branches or pull requests

2 participants