search

dcramer / hass-luxor

FXLuminaire Luxor integration for Home Assistant
MIT License
9 stars 9 forks source link

create separate config instance and change max pool configuration #11

Closed DigitalBites closed 1 year ago

DigitalBites commented 1 year ago

Update the API Client setup to separate out configuration enabling us to change the connection_pool_maxsize. This parameter defaults to 100 in the Luxor API. Given these are little micro-controllers on a landscape lighting controller, this appears to be causing disconnects when too many api calls are made.

Per the Luxor Open API documents, setting this to a small value will queue the requests and send them automatically so this setting should be safe. I've tested this with multiple groups being changed at the same time by adding those Luxor groups to a card, changing the on/off toggle in the header of the home-assistant card and it appears things are updating and working properly.

This Closes #10

DigitalBites commented 1 year ago

@dcramer possible to approve this? Any thoughts?

dcramer commented 1 year ago

🙏 sorry was in my backlog