x10 CM11A support is now fixed!

[jondecker76]

For those of you who have not been able to use their X10 devices, the time is soon coming!

If you want to download the source code, please visit the Trac ticket page here: http://svn.charonmedia.com/trac.cgi/ticket/8

I'm not sure exactly what needs to be done to get this included in the next LMCE, but I'm hoping someone in the know will see this and get the ball rolling for us.

Now for a copy and paste detailing what was fixed, and how to test the new fixes:

Here is the latest version we are calling "release ready".  Please test it. We are going to try to get this into the next version, so we are calling it complete for now. However, please send any bug reports you find, as we are still going to add more to this. This is the current state:

- infamous "checksum" bug fixed
- New routines implemented to fix improper handling of reading incoming data
- Added support for updating the floorplan icons when external remotes are used
- Added Dim/Bright EVENT system into Lighting_Plugin, to include updating of floor plans here too
- Countless bugs fixed in the x10 protocol implementation
- Added support for security devices (x10 motion sensors) and security events

Need to do:
- Nosilla99a is going to find a better, more reliable way to implement IsReadEmpty() in Serial.cpp. Currently, on a fresh reboot, the floorplan will not update when an external remote is used to control an x10 device, until LMCE sends a lighting command. Once this is done though, everything is smooth running. This is looking to be a bug at the kernel level.

-Need to add support to update floorplans when All_Devices_On, All_Devices_Off, and All_Units_Off functions are sent from external remotes. (these functions are normally not available on external remotes anyways)

- Add support for 2-way devices that report their status

- I'm going to thoroughly comment the source code to make it easier for people to update and follow

Also, I'd like to thank the following people, as they were all instrumental in getting this issue fixed:
Nosilla99a - Various code snippets, Checksum bug fix, serial comm stuff, and helped tremendously also by kicking around ideas and figuring out solutions
ddamron - helped out a lot as he had good knowledge of the Lighting_Plugin as he had to learn it for Insteon. Also offered great advice
tkmedia & posde - they were great beta testers and helped test the changes and iron out bugs
hari - helped me get a dev environment set up and gave useful advice
TSCHAK - the guy is a wizard, he saved me countless hours of research by helping me get things accomplished in c++ that I had no idea of how to do

Thanks everyone for the help - I'm going to go to work now on getting this submitted for the next release of LMCE!


If you want to compile the latest versions, all the files needed are in the srcCM11A.tar.gz
/src/CM11A/devicepoll.cpp
/src/CM11A/devicepoll.h
/src/Lighting_Plugin/Lighting_Plugin.cpp
/src/Lighting_Plugin/Lighting_Plugin.h
/src/pluto_main/Define_EventParameters.h
/src/Serial/SerialPort.cpp
/src/Serial/SerialPort.h

To compile: (AT YOUR OWN RISK IF YOU ARE NOT SURE WHAT YOU ARE DOING)
>From CM11A directory:
make bin
sudo rm /usr/pluto/bin/CM11A
sudo cp CM11A /usr/pluto/bin/CM11A

>From Serial directory
make
sudo rm /usr/pluto/lib/libSerial.so
sudo cp libSerial.so /usr/pluto/lib/libSerial.so

>From Lighting_Plugin
make so
sudo rm /usr/pluto/bin/Lighting_Plugin.so
sudo cp Lighting_Plugin.so /usr/pluto/bin/Lighting_Plugin.so

do a force reload router and you're in business.

[golgoj4]

dang. nice work! i'll be testing this later!


edit: or i'll just wait for the update. me + compiling = not fun.

[cobradevil]

Hello,

good job!!!

I have all the problems that seems to be fixed in this code!!
When i turn on the lights i get a checksum error and the cm11a device wil be disabled.
I then have to choose the serial port again but then i also have to enable the child device but when i turn on/off the lights again i have to do all off the above again. So this would be my saving!!!!

But how do i compile the source.
Can someone upload the binaries for the beta2/3 release so we can test this?

With kind regards

William van de Velde

[nite_man]

Great work! Finally the well-known since pluto bug is fixed! Hope we'll see it in the next 0710 beta 3 build.

[Zaerc]

Thanks for doing such a great job on this guys!

[seth]

 
This is great news. Can't wait to get home tonight and try this. My Insteon adapter is still non-functional, but I do have an older IBM Home Director, which is believed to work. I will give this a whirl over the weekend.

Great stuff today!!

Great work everyone!!!

[jondecker76]

Just checked the Mantis ticket I submitted.

The changes have been committed to Revision: 19864

Looks like it should be in for the next release!

[seth]

 
When I cd to /src/CM11A and run
 make bin
It says
 make *** No rule to make target 'bin'.   Stop.

Any particular reason for this. I notice there is no Makefile in the directory.

[jondecker76]

seth -

There are many more files required - those are just the ones that were changed. To compile it, you have to set up a dev environment on your core.

There has been a lot of buzz about another beta coming soon, so I would imagine you won't have to wait very long

[seth]

 
Ah, Ok. That makes more sense. I will wait. Not sure whats required for a Dev environment.

[golgoj4]

Ah, Ok. That makes more sense. I will wait. Not sure whats required for a Dev environment.

this is pretty much why I waved off until an official release. I cant wait to test it though!

[cobradevil]

i compiled the source and it seems to be working!!!!

steps to take for i386 architecture ( i don't have the 64 bit lmce).

make a backup copy off:
cp /usr/pluto/bin/CM11A /root/
cp /usr/pluto/bin/Lighting_Plugin.so /root/
cp /usr/pluto/lib/libSerial.so /root/

then remove those files!!!
rm /usr/pluto/bin/CM11A
rm /usr/pluto/bin/Lighting_Plugin.so
rm /usr/pluto/lib/libSerial.so

download the file http://www.cobradevil.org/downloads/lmce_CM11A_i386_beta2_3.tgz


unpack the file like this:

tar -xzf lmce_CM11A_i386_beta2_3.tgz -C /

- remove the CM11A device from the wall outlet
- reboot (optional) but preferred i guess
- then remove the CM11A interface from the pluto web interface
- plug the CM11A back into the wall outlet
- then recreate that device add some lights
- then reload router and you should be set!!!

With kind regards,

William van de Velde

[golgoj4]

i compiled the source and it seems to be working!!!!

steps to take for i386 architecture ( i don't have the 64 bit lmce).

make a backup copy off:
cp /usr/pluto/bin/CM11A /root/
cp /usr/pluto/bin/Lighting_Plugin.so /root/
cp /usr/pluto/lib/libSerial.so /root/

then remove those files!!!
rm /usr/pluto/bin/CM11A
rm /usr/pluto/bin/Lighting_Plugin.so
rm /usr/pluto/lib/libSerial.so

download the file http://www.cobradevil.org/downloads/lmce_CM11A_i386_beta2_3.tgz


unpack the file like this:

tar -xzf lmce_CM11A_i386_beta2_3.tgz -C /

- remove the CM11A device from the wall outlet
- reboot (optional) but preferred i guess
- then remove the CM11A interface from the pluto web interface
- plug the CM11A back into the wall outlet
- then recreate that device add some lights
- then reload router and you should be set!!!

With kind regards,

William van de Velde

sorry to ask a possibly dumb question, but im guessing your link leads to the compiled files?

[cobradevil]

no dumb questions only dumb answers

Yes it are the precompiled versions.

please test it and report back if possible!

With kind regards,

William van de Velde

[golgoj4]

no dumb questions only dumb answers

Yes it are the precompiled versions.

please test it and report back if possible!

With kind regards,

William van de Velde

sweet! will do tonight