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

[ClockCCSet] after [ClockCCReport] sets device clock back one hour when minutes are rounded up from 59 to 00. #4333

Closed
p3nguin53 opened this issue Mar 8, 2022 · 2 comments · Fixed by #4346 or #4354
Assignees
Labels
bug Something isn't working
Projects

Comments

@p3nguin53
Copy link

Bug occurs when updating time on Aeotec ZWA023 device.

Example of problem from zwave log - 1st ClockCCSet is ok, 2nd ClockCCSet has wrong hour:

2022-03-07T00:59:22.072Z DRIVER « [Node 016] [REQ] [ApplicationCommand]
└─[ClockCCReport]
clock setting: Sunday, 19:00
2022-03-07T00:59:22.074Z CNTRLR [Node 016] detected a deviation of the node's clock, updating it...
2022-03-07T00:59:22.078Z SERIAL » 0x010b001310048104f23b250993 (13 bytes)
2022-03-07T00:59:22.078Z DRIVER » [Node 016] [REQ] [SendData]
│ transmit options: 0x25
│ callback id: 9
└─[ClockCCSet]
clock setting: Sunday, 18:59

2022-03-07T00:59:37.740Z DRIVER « [Node 015] [REQ] [ApplicationCommand]
└─[ClockCCReport]
clock setting: Sunday, 19:00
2022-03-07T00:59:37.741Z CNTRLR [Node 015] detected a deviation of the node's clock, updating it...
2022-03-07T00:59:37.745Z SERIAL » 0x010b00130f048104f200250ab4 (13 bytes)
2022-03-07T00:59:37.745Z DRIVER » [Node 015] [REQ] [SendData]
│ transmit options: 0x25
│ callback id: 10
└─[ClockCCSet]
clock setting: Sunday, 18:00

I found the related section of code in node-zwave-js Node.ts at lines 3081-3130 in the version that I looked at.

If this code is updated to increment the hour when the minutes rounds up from 59 to 00, then it should also set the day to the next day when the time rounds up from 23:59 to 00:00.

I am a newbie to HomeAssistant so I am only reporting this problem and requesting that someone fix this please.
My Zwave Log is in UTC time. I think there is an existing request to display Local Time in this log.

zwave_jslog_4.log

@zwave-js-bot zwave-js-bot added this to Needs triage in Triage Mar 8, 2022
@AlCalzone AlCalzone self-assigned this Mar 8, 2022
@AlCalzone AlCalzone added the bug Something isn't working label Mar 8, 2022
@AlCalzone AlCalzone moved this from Needs triage to Soon™ in Triage Mar 8, 2022
Triage automation moved this from Soon™ to Closed Mar 10, 2022
@p3nguin53
Copy link
Author

p3nguin53 commented Mar 12, 2022 via email

@AlCalzone
Copy link
Member

They won't but thanks for mentioning it. Will put up another PR shortly.

@AlCalzone AlCalzone reopened this Mar 12, 2022
Triage automation moved this from Closed to Needs triage Mar 12, 2022
@zwave-js zwave-js deleted a comment from p3nguin53 Mar 12, 2022
Triage automation moved this from Needs triage to Closed Mar 12, 2022
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
No open projects
Triage
Closed
2 participants