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

babai.curtain.190812 #1019

Closed
wants to merge 13 commits into from
Closed

Conversation

xvlady
Copy link

@xvlady xvlady commented Apr 20, 2021

add my babai.curtain.190812

miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Show resolved Hide resolved
@rytilahti
Copy link
Owner

Also, regarding to that linting error, you can run pre-commit run -a or tox -e lint to run all the linters locally.

miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Show resolved Hide resolved
miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Outdated Show resolved Hide resolved
miio/curtain_babai.py Outdated Show resolved Hide resolved
@xvlady xvlady force-pushed the babai.curtain.190812 branch from aa33835 to 80f4647 Compare May 9, 2021 20:48
rytilahti and others added 12 commits May 10, 2021 10:31
* add miot_info() to get the common information from the device
  * this includes manufacturer, model, firmeware version and serial number

* add get_properties_for_dataclass(cls) which allows easy implementation for get_properties mappings
  * each field() can define metadata containing siid and piid, these are mapped automatically to the response container
  * _siid can be used to define common siid, _max_properties can be used to set number of maximum properties per request

* get_properties_for_mapping() requires explicit passing of the mapping, no more passing over __init__
this is just a poc to show how the new api could function
Co-authored-by: Teemu R. <tpr@iki.fi>
Co-authored-by: Teemu R. <tpr@iki.fi>
Co-authored-by: Teemu R. <tpr@iki.fi>
@xvlady xvlady force-pushed the babai.curtain.190812 branch from 80f4647 to 1bfb460 Compare May 10, 2021 06:46
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