-
Notifications
You must be signed in to change notification settings - Fork 10
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
autodiscovery.py: Change thermostat mode from 'heat' to 'auto' #21
Open
Cymaphore
wants to merge
8
commits into
keton:master
Choose a base branch
from
Cymaphore:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change to cause the Thermostat to report mode 'auto'. This is done by setting auto as the only available mode, using the state topic as mode state topic and providing a static mode state topic enforcing state auto. This way the thermostat is properly visualised and not assumed to be always off.
…ery hour In order to get sensor readings at a more regular interval an option for polling at a certain minute of every hour was added. The polling strategy can be configured (option poll_schedule) and defaults to interval. Option poll_interval now only applies if poll_schedule=interval Option poll_hour_minute accepts a numeric minute For example, when configuring poll_schedule=hour_minute, poll_hour_minute=0 the thermostats will be polled every hour at minute 0. Further options can be added to poll_schedule in future to allow other scheduling strategies (like every two hours or at minute 0,30 or something) or even cron-like config.
Battery, Name and Last Updated report Entity Category 'diagnostic'. Battery and Temperature report State Class 'measurement' to make them available to the statistics system.
Added suggested display precision to improve default display behaviour. temperature: 1 decimal (device resolution is 0.5K) battery: 0 decimals (not an accurate measurement, doesn't require decimals) thermostat: 1 decimal (same as temperature reading, res 0.5K)
…eduler Very weak connections can cause bluepy-helper to semi-crash and stop the scheduler. This commit catches the exception and tries to close the remaining connection (will fail gracefully in case it was completely broken).
Sometimes thermostats with weak connection can be sucessfully polled after a short while. This commit adds the option 'retry_rerun' for such an instance. In case polling retries fail at the first attempt, another attempt is started after finishing all other sensors. The option effectively doubles the number of retry attempts. The option defaults to 'False' for backward compatibility.
Sorry, this file was missed from the previous commit 78912c3
Bad connections can cause bluetooth connections to stay open even when the polling run is finished. Seemingly the valves can be forced to timeout and disconnect by disabling BT alltogether. This commit provides a crude and simple approach by calling rfkill bind/unbind. Not the best way, but seems to work. Marked as experimental due to the lack of long term results (yet). New config option: idle_block_ble (default false) can be used to enable bluetooth disabling between polling cycles.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change to cause the Thermostat to report mode 'auto'. This is done by setting auto as the only available mode, using the state topic as mode state topic and providing a static mode state topic enforcing state auto.
This way the thermostat is properly visualised and not assumed to be always off.
[edit] Additional commit added by push: Configurable polling, schedule for specified minute of every hour