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

Not correct join Gaz Detector SGMHM-I1 and Siren SRHMP-I1 via Zigate in Iobroker... #252

Closed
itProfi opened this issue Nov 8, 2020 · 16 comments

Comments

@itProfi
Copy link

itProfi commented Nov 8, 2020

Good day! I try join device Gaz Detector SGMHM-I1 via Zigate in Iobroker...and have some errors..
How i can correct this situation?
Скриншот 08-11-2020 123830
Скриншот 08-11-2020 124206
Скриншот 08-11-2020 123927
Аfter restart adapter in Iobroker, device like ok, but not work properly...
iobroker.2020-11-08.log

@itProfi
Copy link
Author

itProfi commented Nov 8, 2020

Same problem with smart siren HS2WD-E (it's not "vendor":"HEIMAN", it's russian market Tricolor)
Скриншот 08-11-2020 130842
Скриншот 08-11-2020 130619
iobroker.2020-11-08_siren.log

@Koenkk
Copy link
Owner

Koenkk commented Nov 8, 2020

With what adapter is this?

@itProfi
Copy link
Author

itProfi commented Nov 8, 2020

JN5169 with zigate coordinator firmware via serial port...

@Koenkk
Copy link
Owner

Koenkk commented Nov 8, 2020

The Cannot read property length of undefined indicates an error in the adapter code. @G1K or @kirovilya can you look at this?

@itProfi
Copy link
Author

itProfi commented Nov 9, 2020

@Koenkk In my case, i see, that error is in OTA Update...how i can disable temporary this? I dont shure, vendor is HEIMAN, but for Russian market it name GS..
Скриншот 09-11-2020 142451
smart_siren_OT_error_jon_2.txt

@Koenkk
Copy link
Owner

Koenkk commented Nov 9, 2020

This functionality cannot be disabled currently.

@itProfi
Copy link
Author

itProfi commented Nov 9, 2020

This functionality cannot be disabled currently.

And how i can correct this error? I ready for "dummy" stub....

@Koenkk
Copy link
Owner

Koenkk commented Nov 9, 2020

I expect some fixes are required in the adapter code (#252 (comment))

@itProfi
Copy link
Author

itProfi commented Nov 9, 2020

Today i try only xiaomi gateway with jn5169 and zigbee2mqtt and thats all! I upgrdae zigbee2mqtt to 1.6.1 and give another error -
Скриншот 09-11-2020 191009
Скриншот 09-11-2020 195010

@itProfi
Copy link
Author

itProfi commented Nov 12, 2020

@Koenkk @kirovilya says, that he don't no, what's the problem...last error is
Скриншот 12-11-2020 160440
in Iobroker

@itProfi itProfi changed the title Not correct join Gaz Detector SGMHM-I1 via Zigate in Iobroker... Not correct join Gaz Detector SGMHM-I1 and Siren SRHMP-I1 via Zigate in Iobroker... Nov 12, 2020
@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the stale label Dec 13, 2020
@itProfi
Copy link
Author

itProfi commented Dec 13, 2020

@Koenkk is it possible correct this?

@Koenkk
Copy link
Owner

Koenkk commented Dec 13, 2020

@itProfi I cannot fix this, has to be fixed in the adapter code (#252 (comment))

@itProfi
Copy link
Author

itProfi commented Dec 14, 2020

@Koenkk No, i has lastest code for this adapter and it's not work. @kirovilya says, that he not correct this for at least one more month (( If you want, close this issue..

@G1K
Copy link
Contributor

G1K commented Jan 13, 2021

I am working on this problem. This is where I came to.

  1. We need a separate queue on the APDU, when using a common queue and waiting for a message 0x8000, messages are partially sent to the network.
  2. When pairing, a lot of requests are often generated, for example, on FreePad 20 endpoint + 20 bind. If you use a sequential queue and wait for a message from the network, it works more stable, but still, when using equal requests, the adapter hangs and does not allow messages to be sent to the network. APDU instances are not released fairecasoimeme/ZiGate#348 (This was always the case before firmware 3.1d for incoming messages). The solution is still unknown, I can only suggest rebooting the adapter, but the number of instances for different firmware is different and I have not yet figured out how to find out when they are over.
  3. The fix of expectations from this PR does not wait for an answer, although it is needed and the herdsman wants to get a result. https://github.com/Koenkk/zigbee-herdsman/pull/247/files#diff-12256257d3beef78fe73fdda606b50fbb139127f642a50b74894e21eff30414aR546-R547

As I finish the implementation I will do a PR, while there are only temporary sketches.

// I want to make the most of the adapter, and I don't want to put all messages in the general queue if one of the devices falls asleep and does not receive messages.

@G1K G1K mentioned this issue Jan 13, 2021
7 tasks
@itProfi itProfi closed this as completed Jan 26, 2021
@itProfi
Copy link
Author

itProfi commented Jan 26, 2021

@G1K all do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants