search

zigpy / zha-device-handlers

ZHA device handlers bridge the functionality gap created when manufacturers deviate from the ZCL specification, handling deviations and exceptions by parsing custom messages to and from Zigbee devices.
Apache License 2.0
719 stars 664 forks source link

[BUG] aqara cube knock on faces 2, 3, 5, 6 report wrong activated face #1629

Closed mouth4war closed 2 months ago

mouth4war commented 2 years ago

Describe the bug Doing a double knock on faces 2, 3, 5 and 6 reports incorrect values for activated face:

Below example is for active face 2:

"command": "knock",
"args": {
    "value": 513,
    "description": "aqara logo facing user rotated 90 degrees left",
    "activated_face": 5

To Reproduce Steps to reproduce the behavior:

  1. Enable event listening.
  2. Flip the cube and knock on each face (1-6), check events.
  3. Faces 1 and 4 work fine but knocking on faces 2, 3, 5, 6 generates incorrect activated_face values.

Expected behavior Testing results for knock events: 6 and 3 are interchanged so are 2 and 5. 1 and 4 are reporting correctly.

Screenshots If applicable, add screenshots to help explain your problem.

Device signature ```yaml Paste the device signature here. Don't remove the extra line breaks outside the ``` marks. ```
Diagnostic information ```yaml Paste the diagnostic information here. Don't remove the extra line breaks outside the ``` marks. ```
Additional logs ``` Paste any additional debug logs here. Don't remove the extra line breaks outside the ``` marks. ```

Additional context Add any other context about the problem here.

github-actions[bot] commented 1 year ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

TheJulianJES commented 1 year ago

Relevant PR that also contains some discussion: https://github.com/zigpy/zha-device-handlers/pull/1972

alfa43 commented 12 months ago

Hi, so i got a new Aquqra Cube t1 pro and i got the same issue. How do i resolve this? How do i implement this fix?

github-actions[bot] commented 2 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.