genielabs / HomeGenie

HomeGenie, the programmable automation intelligence
https://homegenie.it
GNU General Public License v3.0
401 stars 157 forks source link

Zwave rework #147

Closed Bounz closed 9 years ago

Bounz commented 9 years ago

Fixes to .csproj and CRC16Encap command class. Handlers folder renamed to CommandClasses.

Bounz commented 9 years ago

Wrong branch.

genemars commented 9 years ago

Hey @hybridview ,

can you help testing current zwave-rework branch?

Thanks, g.

hybridview commented 9 years ago

Sure thing. Could you tell me what the patch 1 and patch 2 folders are for? Just curious.

genemars commented 9 years ago

These should be deleted, just small fixes I've made in place to Bounz code.

hybridview commented 9 years ago

I have tested the new code and everything appears to be working great so far. I had some exceptions at first during startup, but it was related to outdated MIG interface configs in my systemconfig file. After fixing that, no exceptions appear in the console. My testing was mostly focused on automated multi-channel encapped reports (aka multi-instance v2) from my HEMv1 device.

genemars commented 9 years ago

thansk @hybridview, I'm also testing since a couple of days and everything seems ok. I think I'm gonna merge this into master soon.