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

First Alert ZWave Combo [question] #1466

Closed
ragenhe opened this issue Jan 20, 2021 · 32 comments
Closed

First Alert ZWave Combo [question] #1466

ragenhe opened this issue Jan 20, 2021 · 32 comments
Assignees
Labels
bug Something isn't working interview overhaul 🔬 Depends on a reworked interview process

Comments

@ragenhe
Copy link

ragenhe commented Jan 20, 2021

I've converted from Home Assistant native zwave support to zwavejs2mqtt.

So far so good. I hope to contribute to this great project.

Is the First Alert ZWave Combo Smoke / CO2 detector supported?

I have excluded it and included it without issue. I know it can take some time for a battery device to complete the query process, but I'm not sure if it is just taking a while to show it's details or if it is possible not supported.

Thank you!!

@robertsLando
Copy link
Member

@ragenhe Need to see a log in order to tell you what's gouing on :) Go to store and export both zwavejs and zwavejs2mqtt logs

@ragenhe
Copy link
Author

ragenhe commented Jan 20, 2021

I looked through the logs and didn't see anything that jumped out at me for that node #.

I'll give it some time to show up and if not by tomorrow, I will come back and paste them.

@ragenhe
Copy link
Author

ragenhe commented Jan 20, 2021

After further review, I did see an error in the log. The device is node 20.

zwavejs_1.log
zwavejs2mqtt.log

@robertsLando
Copy link
Member

@ragenhe Did you try to manually wake up your device?

@AlCalzone
Copy link
Member

@ragenhe please set the loglevel higher. It seems that a response to something is missing, but I can't figure out why.

@ragenhe
Copy link
Author

ragenhe commented Jan 21, 2021

@robertsLando - I thought I did. Let me retry that.
@AlCalzone - My apologies, I had one at silly and one at info. Both of these are set on silly.

Thanks to you both!
zwavejs2mqtt.log
zwavejs_1.log

@ragenhe
Copy link
Author

ragenhe commented Jan 21, 2021

Update:
I went to node 20 in the UI. Node actions = refresh info/values when I had the device in front of me and manually woke it up.
Here are those results:
Screen Shot 2021-01-21 at 8 47 12 AM

@ragenhe
Copy link
Author

ragenhe commented Jan 21, 2021

Post successful wake up logs.

zwavejs_1.log
zwavejs2mqtt.log

@AlCalzone
Copy link
Member

Okay, we're getting no response to the WakeUpCC interview, so it aborts. I'm going to overhaul the interview procedure so it no longer aborts when we're missing a response, but that will take a bit.

@AlCalzone AlCalzone transferred this issue from zwave-js/zwave-js-ui Jan 21, 2021
@AlCalzone AlCalzone added the bug Something isn't working label Jan 21, 2021
@AlCalzone AlCalzone added the interview overhaul 🔬 Depends on a reworked interview process label Jan 21, 2021
@blhoward2
Copy link
Collaborator

blhoward2 commented Jan 24, 2021

Note that this device has always been terrible to interview under any hub. You generally have to remove the battery and close the door with the button down multiple times to complete the interview process.

There is another thread about issues pairing with this device. Summary:

Pair it as stated in the manual. It will go to sleep before finishing, and HA will report “initializing”.
Pull the battery tray out, wait a few seconds. Push test button and keep depressed, insert battery tray. Release button immediately when the device beeps.
Repeat step 2 until HA reports “sleeping” for the device. At this point, I only had a battery_level sensor.
Restart HA. You should now see _alarm_level and _alarm_type sensors for this node.

https://community.home-assistant.io/t/first-alert-smoke-detector-and-carbon-monoxide-alarm-setup/66761/42?u=blhoward2

@ragenhe
Copy link
Author

ragenhe commented Jan 24, 2021 via email

@blhoward2
Copy link
Collaborator

blhoward2 commented Jan 24, 2021

I didn’t mean that there isn’t a problem here. I should have been clearer. That was a note for @AlCalzone that it may still be abnormal in the logs/procedure.

@ragenhe
Copy link
Author

ragenhe commented Jan 24, 2021

@blhoward2 - I didn't take it that way, no worries at all!!

@AlCalzone
Copy link
Member

@ragenhe please try #1505

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

I updated my dev docker image. I then entered the container and updated via npm install [email protected].

I then manually woke the device up twice. I'll give it a little while and will continue the process.

Thanks!

@AlCalzone
Copy link
Member

if it doesn't work, log please ;)

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

I'm going to continue waking the device and then waiting to see if the end result goal is reached.
Node 021

Screen Shot 2021-01-27 at 9 59 09 AM

But here are the logs so you can see if it is reacting as expected at this point in the process.
zwavejs2mqtt.log
zwavejs_1.log

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

Because this device can be..... is..... difficult, I want to make sure I give you everything I have.

I again manually woke this device up twice and then restarted the container.

No progress towards in result goal. Logs are attached.

zwavejs_1.log
zwavejs2mqtt.log

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

Another manual wake up....

Screen Shot 2021-01-27 at 11 55 01 AM

@AlCalzone
Copy link
Member

Okay, I'll take a look at your logs later

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

The status in the screen shot it the same still. Here are the most recent logs.

No rush! Thanks so much!!!

zwavejs2mqtt.log
zwavejs_1.log

@AlCalzone
Copy link
Member

13:52:01.268 DRIVER   version 6.0.2

@ragenhe doesn't look like your build is actually using the new version.

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

I ran this from within the container
npm install [email protected]

is this the correct approach to update?

@AlCalzone
Copy link
Member

@ragenhe
Copy link
Author

ragenhe commented Jan 27, 2021

I'm on it. Thanks!

@ragenhe
Copy link
Author

ragenhe commented Jan 28, 2021

@AlCalzone - I have followed those directions, but must be missing a step.

I never see the version # get to 6.1.1-0-pr-1505-0102cdc.

@AlCalzone
Copy link
Member

With the custom build you have to set the branch name to use.
You should be able to find that in the PR header, something like "wants to merge branch xyz into master".

@ragenhe
Copy link
Author

ragenhe commented Jan 28, 2021

@AlCalzone - Sorry to be a pain. I attempted git checkout cc-no-response and there isn't one in either repo.

Can you see what I should be using instead of cc-no-responce? Please?

Edit: I've been evaluating the wrong # to compare to what version # I'm after. Sorry, I'm a noob at navigation this project, but glad I'm here. I'm building a new, local image (again) and will let you now what the results are when I'm done testing.

Thanks for all the patience with me!

@AlCalzone
Copy link
Member

Oh yeah things move fast here. Try the master branch 😅

@ragenhe
Copy link
Author

ragenhe commented Jan 28, 2021

@AlCalzone - I finally have an update to report. Using the current (1/28/2021) master branch of both repos, I successfully included the ZCOMBO and with just one additional manual wake up of the device, it was detected 100% and all expected node entries were found on my MQTT broker.

Please let me know if you want to see any of the logs, or need more details.

EDIT: On the HA side I only have a battery_level sensor and a battery_low sensor, but I will continue working on it all until all sensors make it to HA.

Thanks for all the help!!
Ragen

@gregsheremeta
Copy link
Contributor

@ragenhe I got mine to include, but like you I only see the battery sensors in HA. Did you get the actual alarm sensors working in HA?

@ragenhe
Copy link
Author

ragenhe commented Jun 1, 2021 via email

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

No branches or pull requests

5 participants