Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

Initial commit of DSC Alarm Binding #1334

Merged
merged 14 commits into from
Nov 5, 2014
Merged

Initial commit of DSC Alarm Binding #1334

merged 14 commits into from
Nov 5, 2014

Conversation

RSStephens
Copy link
Member

Hello All,

This my first attempt at creating a binding. It is for the DSC Alarm using the EyesOn Envisalink 3/2ds network interface.

Thank you,
Russell

@buildhive
Copy link

openhab » openhab #1044 SUCCESS
This pull request looks good
(what's this?)

* Activation is done there.
*/
public void activate() {
logger.info("Activate DSC Alarm");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please decrease severity to "debug"

@teichsta
Copy link
Member

Hi @RSStephens , thanks for this contribution. I've started the review of your binding and will continue soon. I'll let you know when i am ready. Thanks, Thomas E.-E.

@teichsta teichsta added this to the 1.6.0 milestone Aug 17, 2014
@RSStephens
Copy link
Member Author

Hello Thomas, I have made the changes you requested. Thank you, Russell.

@buildhive
Copy link

openhab » openhab #1073 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

openhab » openhab #1074 SUCCESS
This pull request looks good
(what's this?)

@mrguessed
Copy link
Contributor

You may want to enable TimeDateBroadcastControl at startup and/or reconnect to the TPI. I've found it useful in keeping the connection alive between the Envisalink modules in some of my other work.

For the most part, it means they'll periodically send the time over the wire... but I've seen cases where the connection is artificially dropped if there's no traffic (either by the TPI, or by the other Network components in at CT's environment).

You could also use the "last time" a timestamp was received as a passive indicator that the connection is broken (in openHAB's ActiveBinding polling method), without having to actively reach out to the TPI at the time of the polling event.

@RSStephens
Copy link
Member Author

Mark,

Thanks for the suggestions, I will definitely look into that and do some testing. Currently, I'm sending a POLL command after 15 minutes of idle time just to keep the connection from dropping. Maybe your methods would be more full proof. Thanks again for the insight.

@virtualcrosby
Copy link

Hi @RSStephens

If you need other testers, i would be more then willing. Do you have a wiki page yet for this binding?

@RSStephens
Copy link
Member Author

Hello @bcrosby333

That would be great to have other testers. I have added a wiki page here. I can also send you a sample sitemap if you would like.

Note: This binding is only designed to connect with the EyesOn Envisalink Ethernet interface as that is what I have on my system. It is not capable of connecting to a serial interface.

Thank you,
Russell

@virtualcrosby
Copy link

Hey!

So have been running the binding for a few weeks now. It seems to be stable
as can be. However, i have noticed that the Items do not stay updated and
the binding seems to not be connected at times to my panel.

I have not been able to figure out why yet. What leads me to think this is
the Panel Connected item only rarely will show it is connected; but i will
sometimes (not always) get a Zone status update even though i don't have a
"connections" (based on my items status) to the panel. BUT when i go to
look in the Envisalink webpage...it says my OH server is connected. I can't
seem to correlate when or why the items/panel is connected or not.

For instance, right now the item state for the panel connection is a "-".
But i have a rule for one of my zone alarms (side door) and i am getting
the correct OPEN/CLOSE when i trigger it...

Thoughts?

On Fri, Aug 29, 2014 at 2:27 PM, Russell Stephens notifications@github.com
wrote:

Hello @bcrosby333 https://github.com/bcrosby333

That would be great to have other testers. I have added a wiki page here
https://github.com/RSStephens/openhab/wiki/DSC-Alarm-Binding. I can
also send you a sample sitemap if you would like.

Note: This binding is only designed to connect with the EyesOn Envisalink
Ethernet interface as that is what I have on my system. It is not capable
of connecting to a serial interface.

Thank you,
Russell


Reply to this email directly or view it on GitHub
#1334 (comment).

Brian M Crosby
(703) 286-9128

@RSStephens
Copy link
Member Author

Hello Brian,

Sorry it took so long to get back to you. I just saw your post but somehow
missed your email. I have seen similar issues with the Panel Connection
item not showing the correct status. I will do some testing to try to see
what the problem could be.

Thanks,
Russell Stephens

On Wed, Sep 10, 2014 at 3:44 PM, bcrosby333 notifications@github.com
wrote:

Hey!

So have been running the binding for a few weeks now. It seems to be
stable
as can be. However, i have noticed that the Items do not stay updated and
the binding seems to not be connected at times to my panel.

I have not been able to figure out why yet. What leads me to think this is
the Panel Connected item only rarely will show it is connected; but i will
sometimes (not always) get a Zone status update even though i don't have a
"connections" (based on my items status) to the panel. BUT when i go to
look in the Envisalink webpage...it says my OH server is connected. I
can't
seem to correlate when or why the items/panel is connected or not.

For instance, right now the item state for the panel connection is a "-".
But i have a rule for one of my zone alarms (side door) and i am getting
the correct OPEN/CLOSE when i trigger it...

Thoughts?

On Fri, Aug 29, 2014 at 2:27 PM, Russell Stephens <
notifications@github.com>
wrote:

Hello @bcrosby333 https://github.com/bcrosby333

That would be great to have other testers. I have added a wiki page here
https://github.com/RSStephens/openhab/wiki/DSC-Alarm-Binding. I can
also send you a sample sitemap if you would like.

Note: This binding is only designed to connect with the EyesOn
Envisalink
Ethernet interface as that is what I have on my system. It is not
capable
of connecting to a serial interface.

Thank you,
Russell


Reply to this email directly or view it on GitHub
#1334 (comment).

Brian M Crosby
(703) 286-9128


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@buildhive
Copy link

openhab » openhab #1197 SUCCESS
This pull request looks good
(what's this?)

@RSStephens
Copy link
Member Author

Hello Brian,

I have made some changes to the binding to better monitor the connection to
the panel and update the item status. There were instances where the
connection could drop and the status would not update. Hopefully this
change will make it better. Let me know what you think.

Thanks,
Russell

On Thu, Sep 18, 2014 at 7:45 AM, BuildHive notifications@github.com wrote:

openhab » openhab #1197
https://buildhive.cloudbees.com/job/openhab/job/openhab/1197/ SUCCESS
This pull request looks good
(what's this?) https://www.cloudbees.com/what-is-buildhive


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@virtualcrosby
Copy link

Thanks. I will pull the update tonight and let you know.
On Sep 18, 2014 11:44 AM, "Russell Stephens" notifications@github.com
wrote:

Hello Brian,

I have made some changes to the binding to better monitor the connection
to
the panel and update the item status. There were instances where the
connection could drop and the status would not update. Hopefully this
change will make it better. Let me know what you think.

Thanks,
Russell

On Thu, Sep 18, 2014 at 7:45 AM, BuildHive notifications@github.com
wrote:

openhab » openhab #1197
https://buildhive.cloudbees.com/job/openhab/job/openhab/1197/ SUCCESS
This pull request looks good
(what's this?) https://www.cloudbees.com/what-is-buildhive


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com


Reply to this email directly or view it on GitHub
#1334 (comment).

@mrguessed
Copy link
Contributor

I can't tell, but is itemHasChanged always true in the reconnect() case?

ie. Will it always issue a new StatusReport call to the TPI?

@RSStephens
Copy link
Member Author

Hello Mark,

itemHasChanged *is only set to true when there is a change to any item or
items through a binding change or you receive an internal command. It is
set to true in the *bindingHasChanged()
and the internalReceiveCommand()
methods.

Thanks,
Russell

On Thu, Sep 18, 2014 at 12:28 PM, Mark notifications@github.com wrote:

I can't tell, but is itemHasChanged always true in the reconnect() case?

ie. Will it always issue a new StatusReport call to the TPI?


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@mrguessed
Copy link
Contributor

You may want to force the TPI StatusReport call in the Reconnect case, since you won't be getting internal commands for the duration of the disconnect (that would otherwise trigger that part of the code)

If the disconnect is long-running, then all sorts of changes may have occurred on the DSC. It's probably the biggest complaint I have in the codebase that I wrote for Vera (In my case, I don't have a way of detecting the reconnect due to their internal frameworks).

@RSStephens
Copy link
Member Author

That makes sense. I will implement that. You wrote the interface for DSC
on the Vera? I currently use a Vera 3 for my HA and think it is a great
system. I am also intrigued with the OpenHAB project but it lacked the DSC
binding, so I thought I would give it a shot. I appreciate your input.

Thank you,
Russell

On Fri, Sep 19, 2014 at 1:26 PM, Mark notifications@github.com wrote:

You may want to force the TPI StatusReport call in the Reconnect case,
since you won't be getting internal commands for the duration of the
disconnect (that would otherwise trigger that part of the code)

If the disconnect is long-running, then all sorts of changes may have
occurred on the DSC. It's probably the biggest complaint I have in the
codebase that I wrote for Vera (In my case, I don't have a way of detecting
the reconnect due to their internal frameworks).


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@mrguessed
Copy link
Contributor

The initial Vera DSC binding was written by one of the MCV team, but it was never completed. I re-wrote most of it, along with adding the mods for the Envisalink stuff, and it's the Plugin you see today.

I think a lot of folks in the US will benefit from the DSC binding you're writing, along with the T-Stat (Nest/Ecobee) bindings that watou is converting over.

I still have the Paradox Alarm binding on my todo list, along with a few others (Brultech, etc). I have a working Vera binding which lets me see & control everything on the Vera unit - although, haven't had a lot of cycles lately to push it to review, so it's stalled... gotta get back to that....

@buildhive
Copy link

openhab » openhab #1206 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

openhab » openhab #1207 SUCCESS
This pull request looks good
(what's this?)

@RSStephens
Copy link
Member Author

A Vera binding would be nice to have. I would certainly have a use for it.

On Fri, Sep 19, 2014 at 2:34 PM, Mark notifications@github.com wrote:

The initial Vera DSC binding was written by one of the MCV team, but it
was never completed. I re-wrote most of it, along with adding the mods for
the Envisalink stuff, and it's the Plugin you see today.

I think a lot of folks in the US will benefit from the DSC binding you're
writing, along with the T-Stat (Nest/Ecobee) bindings that watou is
converting over.

I still have the Paradox Alarm binding on my todo list, along with a few
others (Brultech, etc). I have a working Vera binding which lets me see &
control everything on the Vera unit - although, haven't had a lot of cycles
lately to push it to review, so it's stalled... gotta get back to that....


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@buildhive
Copy link

openhab » openhab #1249 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

openhab » openhab #1256 SUCCESS
This pull request looks good
(what's this?)

@automatedrobot
Copy link

thank you for working on the dsc envisalink binding , do you have a example sitemap ?
id like to test an experiment with openhab dsc/envisalink , hope this gets put into the main wiki/repo

@RSStephens
Copy link
Member Author

Hello @automatedrobot,

Glad you want to try this binding. I've added a sample sitemap to the wiki
located here: https://github.com/RSStephens/openhab/wiki/DSC-Alarm-Binding.
It is basically the sitemap I use at home, so you will definitely need to
play around with it. Hope this helps.

Let me know what you think.

Thanks,
Russell

On Tue, Sep 30, 2014 at 11:08 AM, automatedrobot notifications@github.com
wrote:

thank you for working on the dsc envisalink binding , do you have a
example sitemap ?

id like to test an experiment with openhab dsc/envisalink , hope this gets
put into the main wiki/repo


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@buildhive
Copy link

openhab » openhab #1390 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

openhab » openhab #1393 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

openhab » openhab #1394 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

openhab » openhab #1398 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

openhab » openhab #1399 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

openhab » openhab #1400 SUCCESS
This pull request looks good
(what's this?)

@jamjoe
Copy link

jamjoe commented Oct 15, 2014

I have been using this for a couple weeks. It seems to work well. Thanks for writing this binding.

@RSStephens
Copy link
Member Author

Hello @jamjoe,

Your welcome. Glad that you are able to use this binding. Thanks for the
feedback.

Russell

On Wed, Oct 15, 2014 at 1:55 PM, jamjoe notifications@github.com wrote:

I have been using this for a couple weeks. It seems to work well. Thanks
for writing this binding.


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

@buildhive
Copy link

openhab » openhab #1407 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

openhab » openhab #1408 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

openhab » openhab #1410 SUCCESS
This pull request looks good
(what's this?)

@LiberatorUSA
Copy link

I'm hoping to use this binding as soon as I get time to pull an Ethernet cable to my alarm box.
Thanks for all your work in advance!!!

@dk307
Copy link

dk307 commented Nov 3, 2014

I have been using this binding for last few days and has been working flawlessly for couple of weeks. Thanks for the binding.

I ran into some minor issues while setting up the rules. There is no easy way to see if alarm is active or entry/exit delay in progress. I used to have those in Vera, so I missed them for my rules. I ended up using Panel message as input which is not ideal as it requires to look into source code to get the exact strings.

@RSStephens
Copy link
Member Author

Hello @dk307,

Glad you are finding this binding useful. I have been thinking of adding
some trigger type items, basically on/off switches, that would benefit in
rule writing. I will look into it. Thanks for the feedback.

On Mon, Nov 3, 2014 at 4:23 PM, dk307 notifications@github.com wrote:

I have been using this binding for last few days and has been working
flawlessly for couple of weeks. Thanks for the binding.

I ran into some minor issues while setting up the rules. There is no easy
way to see if alarm is active or entry/exit delay in progress. I used to
have those in Vera, so I missed them for my rules. I ended up using Panel
message as input which is not ideal as it requires to look into source code
to get the exact strings.


Reply to this email directly or view it on GitHub
#1334 (comment).

Russell Stephens
RSStephens@gmail.com

teichsta added a commit that referenced this pull request Nov 5, 2014
Initial commit of DSC Alarm Binding
@teichsta teichsta merged commit 703966e into openhab:master Nov 5, 2014
@teichsta
Copy link
Member

teichsta commented Nov 5, 2014

Hi @RSStephens could please also add the Wiki page soon? Please add the link to the page here as well. Thanks, Thomas

@RSStephens
Copy link
Member Author

Thank you Thomas. I've added the Wiki page. Here's the link: https://github.com/openhab/openhab/wiki/DSC-Alarm-Binding.

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

Successfully merging this pull request may close these issues.

9 participants