caronc / ultrasync

API and CLI wrapper to The Interlogix ZeroWire and Hills ComNav (NX-595E) UltraSync Security Panel
MIT License
19 stars 9 forks source link

watch panel dump #7

Closed sbobber closed 3 years ago

sbobber commented 3 years ago

:beetle: Describe the bug

watch panel dump

:bulb: Screenshots and Logs

[fed]$ ./bin/ultrasync -w -c ~/.config/ultrasync/vpnnat.cfg
2021-01-15 14:54:46 [001] F1                      : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] F2                      : Ready
2021-01-15 14:54:46 [001] F3                      : Ready
2021-01-15 14:54:46 [001] P1                      : Ready
2021-01-15 14:54:46 [001] P2                      : Ready
2021-01-15 14:54:46 [001] F4                      : Ready
2021-01-15 14:54:46 [001] F5                      : Ready
2021-01-15 14:54:46 [001] F6                      : Ready
2021-01-15 14:54:46 [001] P3                      : Ready
2021-01-15 14:54:46 [001] P4                      : Ready
2021-01-15 14:54:46 [001] IR1                     : Ready
2021-01-15 14:54:46 [001] IR2                     : Ready
2021-01-15 14:54:46 [001] IR3                     : Ready
2021-01-15 14:54:46 [001] IR4                     : Ready
2021-01-15 14:54:46 [001] IR5                     : Ready
2021-01-15 14:54:46 [001] IR6                     : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] TMP                     : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [001] -                       : Ready
2021-01-15 14:54:46 [081] F                #1     : Armed Away
2021-01-15 14:54:46 [000] B                #2     : Armed Away
---
Traceback (most recent call last):
  File "./bin/ultrasync", line 71, in <module>
    main()
  File "/home/xxx/.local/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/xxx/.local/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/xxx/.local/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/xxx/.local/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/xxx/04_configurazioni/ultrasync_caronc/ultrasync-xgen-nx64ip-support/ultrasync/cli.py", line 199, in main
    results = usync.details()
  File "/home/xxx/04_configurazioni/ultrasync_caronc/ultrasync-xgen-nx64ip-support/ultrasync/main.py", line 588, in details
    if not self.update(max_age_sec=max_age_sec):
  File "/home/xxx/04_configurazioni/ultrasync_caronc/ultrasync-xgen-nx64ip-support/ultrasync/main.py", line 573, in update
    if not self._sequence():
  File "/home/xxx/04_configurazioni/ultrasync_caronc/ultrasync-xgen-nx64ip-support/ultrasync/main.py", line 1829, in _sequence
    return getattr(self, '_{}_sequence'.format(self.vendor))()
  File "/home/xxx/04_configurazioni/ultrasync_caronc/ultrasync-xgen-nx64ip-support/ultrasync/main.py", line 2046, in _comnav_sequence
    if area['sequence'] != a_seq[bank]:
IndexError: list index out of range

:computer: Your System Details:

:crystal_ball: Additional context

caronc commented 3 years ago

@sbobber,

I can definitely have a look into this for you! :slightly_smiling_face: . I also noticed you're using the ultrasync-xgen-nx64ip-support branch. Is that intentional?

What i'll need from you is a ultrasync --debug-dump call from you. It will produce a .zip file with everything i need about your security panel (don't worry, there is no pin and/or password information in this dump). It will however share with me your zones/areas. I need this so i can re-produce your situation though and attempt to resolve it.

Would you be able to do that for me and pass the zip file back through this ticket? If you have any privacy concerns just email it to me directly (@ lead2gold@gmail.com).

Chris

sbobber commented 3 years ago

Hi, the problem is the same in all version, I've opened the issue with the latest release, do you preferer the dump with the stable one?

caronc commented 3 years ago

Whatever version you have will work great; they all produce the same file. I was only asking about the branch thinking that maybe you also had an xGen panel too. If your panel is Interlogix or ComNav then it really doesn't matter where the dump comes from :slightly_smiling_face:

caronc commented 3 years ago

@sbobber,

Just letting you know i got the dump file; thank you! I'll hopefully have a look at it this weekend. :+1:

caronc commented 3 years ago

I think i might have fixed this. I got your issue mixed up with another which is why the branch naming is off. But if you want to check out the 8-comnav-all-zone-fix branch and give it a try. It might fix things for you.

There was a MAJOR flaw in ALL my releases of ultrasync that you uncovered in how i was interpreting bank information. I thought the bank information mapped 1:1 to the Area in question and they do not; not at all. I rewrote how sequence information is tracked completely. I'm hoping this will rectify the issue you're having (i may have introduced more as this is a pretty significant change).

Please test it out and let me know.

sbobber commented 3 years ago

the dump is resolved, but something strange appear, seem that the -w flag check only the first area, but let me do more test

caronc commented 3 years ago

By chance are the entries you have defined as a dash - valid? I thought these were invalid and added them to the ignore list. Should I undo this?

caronc commented 3 years ago

Hi, i haven't heard back from you. I've gone ahead and merged the branch since it resolved your issue. I left the - zone entries as to be ignored. If this isn't correct, just let me know.

sbobber commented 3 years ago

I tried 0.9.3, now work like charm, thanks