Open phiz118 opened 3 years ago
I was just looking at this myself as our system has cray numbering and the last station is station 17 despite having only 11 zones. If you change line 223 of _ _ini.py from lambda resp: States((mask % resp["activeStations"])[:4]), to lambda resp: States((mask % resp["activeStations"])[:6]), and line 92 of data.py from def init(self, mask="0000"): def init__(self, mask="000000"): this fixes the issue on my Home Assistant setup
@idoc69 would you be able to enable debug logging and record the response from the server with the active stations in it? I think that would increase our confidence about ability to have tests for this case to ensure it works going forward.
I currently have 18 zones at my home. Rainbird controllers can support 22 stations. Under issue #15, the limit was 8 and they requested it updated to 16, but unsure if that change caused the limit. If so - would you mind updating the limit to 22? Here's the error I see in the logs. Home Assistant is loading the first 16 stations then I believe throwing this error that it failed due to the index out of range on self.states.