Skip to content
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

lux devices, lux devices complex, and lux devices simple commands have same output #35

Closed
ghost opened this issue May 1, 2022 · 1 comment

Comments

@ghost
Copy link

ghost commented May 1, 2022

I noticed when trying to get some additional information about each of my devices that the lux devices... commands all output the same information. See below for the output of all three commands on my installation.

PS C:\Users\Beau> lux devices
DEVICE 0
  Name:           East bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 1
  Name:           Bedroom B
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 2
  Name:           Bedroom A
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 3
  Name:           North bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 4
  Name:           South-east bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 5
  Name:           South-west bulb
  Model:          H6008
  Controllable:   [ Yes]
PS C:\Users\Beau> # lux devices complex output

PS C:\Users\Beau> lux devices complex
DEVICE 0
  Name:           East bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 1
  Name:           Bedroom B
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 2
  Name:           Bedroom A
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 3
  Name:           North bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 4
  Name:           South-east bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 5
  Name:           South-west bulb
  Model:          H6008
  Controllable:   [ Yes]
PS C:\Users\Beau> # lux devices simple output

PS C:\Users\Beau> lux devices simple
DEVICE 0
  Name:           East bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 1
  Name:           Bedroom B
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 2
  Name:           Bedroom A
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 3
  Name:           North bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 4
  Name:           South-east bulb
  Model:          H6008
  Controllable:   [ Yes]
DEVICE 5
  Name:           South-west bulb
  Model:          H6008
  Controllable:   [ Yes]

I don't believe this is the intended output for these commands, except for perhaps the "simple" version. If this is a work-in-progress feature that hasn't been fully implemented and I missed where that was noted, please accept my apologies.

Lux instance info:

  logged in       [✓ Yes]
  description     Lux is a cli for controlling and monitoring Govee lighting strips
  build           v1.1.1 (x64)
  license         GPL-3.0
  repository      https://github.com/bandev/lux
  up to date      [✓ Yes]
@jackdevey
Copy link
Owner

I changed the parameters for the command a while back when moving to flags but never actually updated the help page and documentation!

lux devices --expert

Should show all the data about the device:

DEVICE 0
  MAC Address:    F5:75:A4:C1:38:7E:59:6C
  Model:          H6159
  Name:           Desk Lights
  Controllable:   [✓ Yes]
  Retrievable:    [✓ Yes]
  Commands:       turn, brightness, color, colorTem

Whereas

lux devices

Should just show the simple device data

DEVICE 0
  Name:           Desk Lights
  Model:          H6159
  Controllable:   [✓ Yes]

I will update the documentation now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant