search

MariusRumpf / node-lifx

Node.js implementation of the LIFX LAN protocol :bulb:
MIT License
144 stars 28 forks source link

getColorZones doesn't return all zones. #57

Open Einstein42 opened 7 years ago

Einstein42 commented 7 years ago

getColorZones states a count of 16 zones. Indexes set to 0, 15, it only returns first 8 HSBK items no matter if I ask for 2 zones or all 16.

  light.getColorZones(0, 15, (err, data) => {
    logger.debug(JSON.stringify(data))
  })
{ size: 102,
  addressable: true,
  tagged: false,
  origin: true,
  protocolVersion: 1024,
  source: 'e6b8b4ac',
  target: 'd073d5149b32',
  reserved1: <Buffer 00 00>,
  site: 'LIFXV2',
  ackRequired: false,
  resRequired: false,
  sequence: 6,
  time: <Buffer 48 90 5e 01 7f 92 c9 14>,
  type: 'stateMultiZone',
  reserved2: <Buffer 00 00>,
  count: 16,
  index: 0,
  color: 
   [ { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 },
     { hue: 9102, saturation: 65535, brightness: 65535, kelvin: 3500 } ] }
ristomatti commented 7 years ago

@Einstein42 I had some issues with this while adding the functionality but if I remember correctly I got it working myself. I can try this out in a few days when I'm back home.