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

Add temperature offsets #137

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CSchlipp
Copy link

@CSchlipp CSchlipp commented Feb 16, 2025

This PR adds two optional configuration values in the yaml for outside & current temperature offsets.

current_temperature_offset: 0
outside_temperature_offset: 0

The offsets are added to the temperature readings and internally subtracted from the target temperature settings.
Examples:

  • If the temperature is actually higher than measured by the AC, set the difference as a positive offset.
    • E.g. actual temperature = 23°, AC measured temperature = 20° --> offset = 3°
      • current temperature is reported as 23°
      • target temperature sent to the AC is reduced by 3°, so that the AC is not heating beyond actual temperature
        • target temperature set to 23° will internally be sent as target = 20° to the AC (which corresponds to the actual temperature of 23°)
  • If the temperature is actually lower than measured by the AC, set the difference as a negative offset.
    • E.g. actual temperature = 20°, AC measured temperature = 22° --> offset = -2°
      • Current temperature is reported as 20°
      • target temperature sent to the AC is reduced by -2° (=increased by 2°), so that the AC heats more to reach the actual temperature
        • target temperature set to 20° will internally be sent as target = 22° to the AC (which corresponds to the actual temperature of 20°)

Tested only with CNT variant.

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.

1 participant