Re: [mh] Insteon PLM (Insteon Protocol Support) added

[Jason S. <ja...@sh...>]

On Fri, 28 Sep 2007, Gregg Liming wrote:

> Hi Jason,
>
> Quoting Jason Sharpee (9/27/07 7:46 AM):
>
>> However, I am not translating the status_request result.  I tried it out
>> just now on my test device and can easily see where it is responding with
>> a command1 '00' and a command2 with a hex level '00-FF'.  Should be no
>> more than a 5 minute change to the code and ill see if I can add that this
>> weekend as well.
>
> While you're making mods, would you consider applying the attached diff
> (or something of comparable outcome) to Insteon_Device so that instances
> "play nice" w/ your Light_Item.pm?  Note that I have m_write default to
> 1 so that "transmit only" instances would have to: $obj->writable(0);
> I'm not opposed to defaulting the other way so long as writable exists.
>
> On a related note, do you have plans for addressing/implementing the
> Insteon-specific (non-x10) ALL-Linking capabilities?  I'm quite
> interested in anything along these lines and would be happy to
> contribute (code, test, etc.) anyway that would help.
>

Gregg, thanks for the patch (I tossed too much out on my cleaup).  I 
checked in support for receiving status requests, cleaned up the debug 
messages, and added your patch.

As I am not an Insteon user myself so I only intended the Insteon code to 
kick off efforts in this realm by others that do use it.  I am not 
planning on making any more changes to that code, so anyone is welcome to 
make changes directly in SVN.  If anyone has questions on the 
implementation feel free to contact me, however, I am going to start 
looking into an xAP device type.

Once again, I would like to thank everyone for their help in getting this 
up and running!

-J