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 oscillation to fan #594

Merged
merged 8 commits into from
Feb 5, 2021
Merged

Add oscillation to fan #594

merged 8 commits into from
Feb 5, 2021

Conversation

crazyfx1
Copy link
Contributor

@crazyfx1 crazyfx1 commented Feb 4, 2021

Description

HA supports turning the oscillation of the fan on or off. https://developers.home-assistant.io/docs/core/entity/fan/
Furthermore in #466 there was the request to control the oscillation via switch DPT 1.001

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • I have performed a self-review of my own code
  • The documentation has been adjusted accordingly
  • The changes generate no new warnings
  • Tests have been added that prove the fix is effective or that the feature works
  • The changes are documented in the changelog
  • The Homeassistant plugin has been adjusted in case of new config options

@coveralls
Copy link

coveralls commented Feb 4, 2021

Coverage Status

Coverage increased (+0.01%) to 94.871% when pulling 35e7b09 on crazyfx1:feat/fan_oscillate into aa09383 on XKNX:master.

docs/fan.md Outdated Show resolved Hide resolved
docs/fan.md Outdated
- `group_address` is the KNX group address of the fan speed. Used for sending.
- `group_address_state` is the KNX group address of the fan speed state. Used for updating and reading state.
- `group_address_oscillation` is the KNX group address of the oscillation. Used for sending. *DPT 1.001*
- `group_address_oscillation_state` is the KNX group address of the fan oscillation state. Used for updating and reading state. *DPT 1.001*
- `device_updated_cb` awaitable callback for each update.
- `max_step` Maximum step amount for fans which are controlled with steps and not percentage.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a note like "if None percentage is used" and "Default: None" so it is clear when DPT 5.001 or 5.010 is used.

xknx/devices/fan.py Outdated Show resolved Hide resolved
Copy link
Member

@farmio farmio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great!

@farmio farmio merged commit d4d3b47 into XKNX:master Feb 5, 2021
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.

3 participants