-
-
Notifications
You must be signed in to change notification settings - Fork 540
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
Initial support for HUIZUO PISCES For Bedroom #868
Conversation
2. Changing "color_temp" instead of "level" during manipulation with color temperature
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A quick review, this looks very straight forward, great work! Please run pre-commit run --all
which will fix the linting issues automatically for you.
miio/huizuo.py
Outdated
|
||
class HuizuoStatus: | ||
def __init__(self, data): | ||
# power = '{"siid":2,"piid":1}' values = true,false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add the complete response payload (miiocli -d
) here for future reference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added in the file as comments (is this what you asked for? ;))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, sorry, just the json payload please, here's a good example: https://github.com/rytilahti/python-miio/blob/master/miio/airdehumidifier.py#L59
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it.
Updated by example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also do not forget to update the README file and add a couple of tests!
Updated README. With the tests I have some troubles, can't get glue how to make it work, even test_get_status method. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Adding Huizuo basic support * Add huizuo.py to the repo * Update __init__.py to the latest version * Fix _LOGGER error * Enabling iSort in VSCode on Save * 1. Removed unnecessary click.argument calls 2. Changing "color_temp" instead of "level" during manipulation with color temperature * 1. Removed Huizuo from discovery - devices are not mdns discoverable 2. Updated based on PR#868 * Re-arranged color_temp parameter for better understanding * fixing linting issues * Added only example of JSON payload from the lamp * Updated README.rst, added separations * 1. Use MiotDevice class 2. Add tests * Fixing linting issues * Fixing linting issue - second try... * Processing comments from PR#868
I have such a lamp and wrote the basic library to support it. Please review.