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

Arming IQ Pro to Night throws an error #160

Open
2 tasks done
alistairg opened this issue Aug 4, 2024 · 1 comment
Open
2 tasks done

Arming IQ Pro to Night throws an error #160

alistairg opened this issue Aug 4, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@alistairg
Copy link

Before submitting this form

  • I have checked that there hasn't been a similar issue submitted in the past

  • I have updated the issue title to something relevant and clear to understand the bug I'm facing

Description of the bug

When arming to night an error is raised. Checking in the code it looks like support for ARM_NIGHT has been commented out - before I start to attack this, I wanted to see if this was because of some other issue, or simply not yet implemented?

Expected behavior

Home assistant status should reflect the armed_night nature of the panel

DEBUG logs

Traceback (most recent call last):
File "/config/apps/qolsysgw/mqtt/listener.py", line 47, in event_callback
await self._callback(event)

File "/config/apps/qolsysgw/gateway.py", line 210, in mqtt_event_callback
self._state.update(event)

File "/config/apps/qolsysgw/qolsys/state.py", line 56, in update
self.notify(change=self.NOTIFY_UPDATE_PARTITIONS,

File "/config/apps/qolsysgw/qolsys/observable.py", line 23, in notify
callback(self, **payload)

File "/config/apps/qolsysgw/mqtt/updater.py", line 57, in _state_update
self._factory.wrap(partition).configure()

File "/config/apps/qolsysgw/mqtt/updater.py", line 218, in configure
self.update_state()

File "/config/apps/qolsysgw/mqtt/updater.py", line 426, in update_state
payload=self.ha_status,
^^^^^^^^^^^^^^

File "/config/apps/qolsysgw/mqtt/updater.py", line 342, in ha_status
raise ValueError('We need to put a better error here, but '
ValueError: We need to put a better error here, but we found an unsupported status: 'ARM_NIGHT'

Additional context

No response

@alistairg alistairg added the bug Something isn't working label Aug 4, 2024
@XaF
Copy link
Owner

XaF commented Sep 9, 2024

When arming to night an error is raised. Checking in the code it looks like support for ARM_NIGHT has been commented out - before I start to attack this, I wanted to see if this was because of some other issue, or simply not yet implemented?

The main issue is not being able to identify which panel is what, and so which real set of features are available. To my knowledge, no panel API allows to know the current panel version, and this was built around an IQ Panel 2+ initially. If the IQ Pro supports night, support should be able to be added. Ideally, we'll want to add some configuration option around if arm night should be available or not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants