Danielhiversen / flux_led

A utility for controlling Flux WiFi Smart LED Light Bulbs
GNU Lesser General Public License v3.0
156 stars 144 forks source link

Query device config for a1/a2 on connect #270

Closed bdraco closed 2 years ago

bdraco commented 2 years ago

Supports #267

bdraco commented 2 years ago
2021-12-24 15:47:44 DEBUG (MainThread) [flux_led.base_device] 192.168.106.241: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=30, segments=10, music_pixels_per_segment=30, music_segments=10, wiring='RGB', protocol='WS2811', operating_mode='RGB')
2021-12-24 15:47:44 DEBUG (MainThread) [flux_led.base_device] 192.168.106.102: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=50, segments=0, music_pixels_per_segment=0, music_segments=0, wiring='GRB', protocol='WS2812B', operating_mode='RGB')
2021-12-24 15:47:44 DEBUG (MainThread) [flux_led.base_device] 192.168.106.118: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=25, segments=2, music_pixels_per_segment=25, music_segments=2, wiring='GBR', protocol='WS2811', operating_mode='RGB')
2021-12-24 15:47:44 DEBUG (MainThread) [flux_led.base_device] 192.168.107.144: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=11, segments=10, music_pixels_per_segment=30, music_segments=10, wiring='GBR', protocol='SM16703', operating_mode='RGB')
2021-12-24 15:47:44 DEBUG (MainThread) [flux_led.base_device] 192.168.106.28: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=144, segments=1, music_pixels_per_segment=144, music_segments=1, wiring='WGRB', protocol='SK6812RGBW', operating_mode='RGBW')
2021-12-24 15:47:45 DEBUG (MainThread) [flux_led.base_device] 192.168.106.179: device_config: LEDENETAddressableDeviceConfiguration(pixels_per_segment=50, segments=1, music_pixels_per_segment=50, music_segments=1, wiring='BRG', protocol='WS2812B', operating_mode='RGB')

Working well for A1,A2,A3

Need to write tests and work on 0x07,0x06,0x08,0x33,0x25

codecov-commenter commented 2 years ago

Codecov Report

Merging #270 (706a2ab) into master (1ace807) will increase coverage by 0.57%. The diff coverage is 99.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #270      +/-   ##
==========================================
+ Coverage   85.58%   86.16%   +0.57%     
==========================================
  Files          15       15              
  Lines        2546     2645      +99     
==========================================
+ Hits         2179     2279     +100     
+ Misses        367      366       -1     
Impacted Files Coverage Δ
flux_led/base_device.py 97.80% <98.48%> (+0.08%) :arrow_up:
flux_led/aiodevice.py 100.00% <100.00%> (ø)
flux_led/models_db.py 100.00% <100.00%> (ø)
flux_led/protocol.py 98.48% <100.00%> (+0.61%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1ace807...706a2ab. Read the comment docs.