I’m just trying to control my smartlights with a script, it seems to be having a lot of problems, I really don’t know what I’m doing, i’d appreciate any help I can get

Once I have a script that can individually turn lights on/off i can edit the rest myself, I just can’t get the base functionality working.

  • @xantoxis
    link
    7
    edit-2
    8 months ago

    Light, apparently a class in that library, has not been defined, or imported. In Python, whenever you reference any expression, such as Light(...) on the right-hand-side of an assignment, it must already be present in the program (technically, in this case: the global namespace).

    Not entirely your fault, I quickly glanced at the library you’re using and in several places the examples use Light() the same way you did, but without importing it first. This is… not unreasonable in documentation, as you’re expected to have some familiarity with Python, but they don’t really tell you where to import it from.

    Another quick glance at the source code suggests that either of these should work, and do the intended thing:

    1. Add: from lifxlan import Light near the top
    2. Replace your import line with, simply, import lifxlan, and then use lifxlan.Light(...) and lifxlan.LifxLAN() where those names appear in the program.[1]

    I strongly suggest taking a few hours to go through a Python tutorial, as you’re going to quickly get into more advanced stuff as you try to get this working. It will help to have a framework of understanding before you do.

    [1] Note: the line lan = LifxLAN() probably isn’t doing anything at all in this program, but I assume you’ll be adding more code later that uses it.

    • CommunistOP
      link
      fedilink
      English
      48 months ago

      I just want you to know I figured it out RIGHT BEFORE you wrote this, but this is still awesome and super helpful and thank you so much!

  • [email protected]
    link
    fedilink
    68 months ago

    Glad you got it figured out.

    Tangent: Something like home assistant could also abstract a fait bit of complexity especially when you start dealing with different manufacturers.
    Just checked and they already support lifx.
    Good luck