mjg59 / python-broadlink

Python module for controlling Broadlink RM2/3 (Pro) remote controls, A1 sensor platforms and SP2/3 smartplugs
MIT License
1.35k stars 477 forks source link

Update hub.py #754

Closed stevendodd closed 2 months ago

stevendodd commented 1 year ago

When you add and remove devices from the hub the hub can return more than the total devices..

Context

Proposed change

Type of change

Additional information

Checklist

felipediel commented 2 months ago

Thank you @stevendodd! It seems that the dids end up being duplicated when the order changes. How about we add a filter to avoid extending sub_devices with duplicate dids?

felipediel commented 2 months ago

Closed with https://github.com/mjg59/python-broadlink/pull/790. Thanks @stevendodd!