multizone - line 43

[ceboxsell]

The Multizone code responds with

for i in range(((upper_8_aligned - lower_8_aligned) / 8) + 1):
TypeError: 'float' object cannot be interpreted as an integer

Needs int division operator so for i in range(((upper_8_aligned - lower_8_aligned) // 8) + 1): works.

[mclarkk]

Looks like you are using floats instead of ints to specify the start/end indices when calling get_color_zones(). Indices can only be integers.

[ceboxsell]

No, I am using your examples and know little about Python but was trying to keep it short but found a problem. From: ML Clark [mailto:notifications@github.com] Sent: Saturday, 24 June 2017 2:51 AM To: mclarkk/lifxlan <lifxlan@noreply.github.com> Cc: Chris Boxsell <chris@boxsell.net>; Author <author@noreply.github.com> Subject: Re: [mclarkk/lifxlan] multizone - line 43 (#54) Looks like you are using floats instead of ints to specify the start/end indices. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#54 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AJqqFJazG6KSGqGmd4a5Nn5z5wvzS3AEks5sG-0LgaJpZM4N_hmE> . <https://github.com/notifications/beacon/AJqqFCj9cbcaTROJJNDe5olmUGtkN8WXks5sG-0LgaJpZM4N_hmE.gif>

[mclarkk]

Could you give me information about what program you were running? I don't have enough information here to troubleshoot.

[mclarkk]

I see that all the multizone example programs have this error when run with Python3.

In the future, there is no need to keep a bug report short. In general, the more information you provide the better!

[mclarkk]

All fixed! If you installed lifxlan from source, you can pull from the repo and re-run sudo python setup.py install, and if you installed from pip, I pushed a new version there so you can update it that way too! Thanks for letting me know about this issue!