veista / nilan

Nilan integration for Home Assistant
Apache License 2.0
45 stars 10 forks source link

Set VGU180 EK supported registers #107

Closed jyrkij closed 7 months ago

jyrkij commented 7 months ago

This is based on the support matrix on register group level found at https://www.en.nilan.dk/Files/Files/Engelsk/Downloads/7.%20Modbus%20-%20BACnet/2021-01-Modbus_CTS602_w_HMI350T_Modbus.pdf. All registers in a group might not be relevant for this device but are still "supported". Resolves #105.

veista commented 7 months ago

Unless you cross-check that the registers actually function, I won't have it. All registers are "supported" after modbus version 22.

jyrkij commented 7 months ago

Is there a way to disable registers that are supported by “all” from single device?

veista commented 7 months ago

Yes. Is there some "all" register that does not work on your device?

jyrkij commented 7 months ago

e.g. the supply air side. There is no supply air physically present.

jyrkij commented 7 months ago

I did (force) push again to the branch. I believe I've gone through almost all the registers and verified if they work or not, here's the breakdown

Would you like a new pull request or reopening this one?

veista commented 7 months ago

You can reopen this one.

jyrkij commented 6 months ago

…I might be missing a permission to reopen this one as the button to do so isn’t there…

veista commented 6 months ago

For some reason I can't re-open this either. Just make a new one.