-
-
Notifications
You must be signed in to change notification settings - Fork 561
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
Exception when displaying Power load using Plug CLI #144
Comments
Currently only powerstrip offers load_power, I think the cli tool should be modified to detect what type of plug device it's accessing and behave accordingly. Similar to what is done in https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/switch/xiaomi_miio.py . |
Ain't the Plug CLI only be used for plugs? If it's only for plugs then we don't really have to care about detecting its type for now. |
It would be nice to be able to control also e.g. that powerstrip with the same tool for two reasons: 1) I have not find time to create a generic tool allowing control of all supported devices, and won't be able to do so in the near future, and 2) the interface for plugs and that of the strip share many things in common, so supporting both would not be a big hurdle. |
Problem: Running the Plug CLI will cause an exception on
plug_cli.py:81
when displaying Power load due toPlugStatus.load_power()
not available.Suggestion: get rid of this line or implement
PlugStatus.load_power()
The text was updated successfully, but these errors were encountered: