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

FeatureRequest: HVACStatus not handle Heat/Cool bit #1348

Closed
bussnet opened this issue Oct 25, 2023 · 1 comment · Fixed by #1500
Closed

FeatureRequest: HVACStatus not handle Heat/Cool bit #1348

bussnet opened this issue Oct 25, 2023 · 1 comment · Fixed by #1500
Labels
💡 feature request Feature request

Comments

@bussnet
Copy link

bussnet commented Oct 25, 2023

As defined in the KNX Standard Appendix A (defined here as KNX AN 097/07 rev 3) contains the heat/cool status in the b5 bit, but is not interpreted.

    def from_knx(cls, payload: DPTArray | DPTBinary) -> HVACOperationMode:
        """Parse/deserialize from KNX/IP raw data."""
        raw = cls.validate_payload(payload)
        if raw[0] & 8 > 0:
            return HVACOperationMode.FROST_PROTECTION
        if raw[0] & 4 > 0:
            return HVACOperationMode.NIGHT
        if raw[0] & 2 > 0:
            return HVACOperationMode.STANDBY
        if raw[0] & 1 > 0:
            return HVACOperationMode.COMFORT
        raise ConversionError(f"Payload not supported for {cls.__name__}", raw=raw)

This is important, because this is the only way to receive the HEAT/COOL state of an MDT HeatAktor (AKH-0800.03)

I think the HVACOperationMode must be restructured, because the HVAC Status return multiple values, not only one.
A workaround could be to use the COOL/HEAT bit from the status and affect the controller_mode.

KNX Spec for this HVAC Status: 03_07_02 Datapoint Types v01.08.02 AS.pdf

@farmio
Copy link
Member

farmio commented Oct 25, 2023

The actuator you mentioned, like most current models, provide simple DPT 1 objects for setting and reading heat/cool status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 feature request Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants