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 attributes for the AirPurifier #419

Merged
merged 6 commits into from
Feb 5, 2022
Merged

Add attributes for the AirPurifier #419

merged 6 commits into from
Feb 5, 2022

Conversation

ggravlingen
Copy link
Member

This PR adds attributes that are available on the AirPurifier fan. Some information regarding the properties have been gathered from this post: dresden-elektronik/deconz-rest-plugin#5351 (comment)

pytradfri/const.py Show resolved Hide resolved
pytradfri/device/air_purifier.py Outdated Show resolved Hide resolved

@property
def filter_status(self) -> int:
"""Return filter status."""
Copy link
Contributor

Choose a reason for hiding this comment

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

What does this integer represent in terms of filter status?

Copy link
Member Author

Choose a reason for hiding this comment

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

I am not sure to be honest as I haven't had my filter for more than three months (its lifetime seems to be six months). If I were to make a guesss it will return 1 when its time to replace the filter. But I'm not willing to put that into the documentation of the property yet.

Copy link
Contributor

Choose a reason for hiding this comment

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

Does it return 0 today for you?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes:

{'5900': 1,
 '5902': 152245,
 '5903': 0,
 '5904': 259200,
 '5905': 0,
 '5906': 1,
 '5907': 8,
 '5908': 10,
 '5909': 152244,
 '5910': 106955,
 '9003': 0}

@ggravlingen ggravlingen merged commit b669a9f into master Feb 5, 2022
@ggravlingen ggravlingen deleted the add-properties branch February 5, 2022 11:09
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.

2 participants