search

deennoo / domoticz-Xiaomi-Led-Lamp

Domoticz Plugin for Philips ZhiRui E27 bulb aka Xiaomi Philips LED Ball Lamp
9 stars 2 forks source link

Error on level 100 #6

Open albebert opened 5 years ago

albebert commented 5 years ago

i found an error.

when i try to set level to 100 the call failed

Error: (spot) Call command failed: Command '['./MyBulb.py', '192.168.201.72', '654f182f6b98342ed15b1de364717ae5', '--level', '100']' returned non-zero exit status 2

in fact the python lib andle level vrem 0 to 99 not 100

MyBulb.py: error: argument --level: invalid choice: 100 (choose from 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99)

SamaelRanger commented 5 years ago

Good caught! Fixed.

SamaelRanger commented 5 years ago

@albebert please test the latest changes.

albebert commented 5 years ago

Seems to work.

Just à Little question, do you think that is possible to modify the script in order to drive multiple Bulb with one domoticz device,

I want to threat 6 spot on m'y living room, I know that dz groups arent good due to miio lib. But I think that if device are drive at same time with fork this Will work.

I know that I have only 1 Bulb status but not à problem for me.

What do you think about this ?

SamaelRanger commented 5 years ago

Yes, sure. You can modify the script and append another IP addresses of bulbs.

пн, 25 мар. 2019 г. в 01:44, albebert notifications@github.com:

Seems to work.

Just à Little question, do you think that is possible to modify the script in order to drive multiple Bulb with one domoticz device,

I want to threat 6 spot on m'y living room, I know that dz groups arent good due to miio lib. But I think that if device are drive at same time with fork this Will work.

I know that I have only 1 Bulb status but not à problem for me.

What do you think about this ?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/deennoo/domoticz-Xiaomi-Led-Lamp/issues/6#issuecomment-476008810, or mute the thread https://github.com/notifications/unsubscribe-auth/AG7SsXBEBpGzGplB9Rz0qKkG_xf1DFJWks5vZ__qgaJpZM4bVpBp .

-- Eugene.

albebert commented 5 years ago

ok need to learn some python ^^

test with the miiocli, some delay due to high cpu usage.

tested on 4 spot simultaneously, on my pi 2, all CPU are at 100% and some delay between power off/on (less than 1s)

if i limit to 2 spot simultaneously work like a charm.

the target is 6 or 7 spots :/