search

jllcunha / openhab-greeair-binding

A openHab2 binding for Gree Airconditioners
Apache License 2.0
17 stars 13 forks source link

Update README.md #5

Open chutch opened 5 years ago

chutch commented 5 years ago

Hey, I've struggled a bit with this binding and the problem was in configuration. Here's PR to spare some time for others. :-)

pezoli commented 5 years ago

Hi Chutch, why do you need to send the broadcast messages to another subnet, not the one your Gree unit resides on? I am far from being expert, just would like to understand, because i didn't specify the subnet in Thing config file, still everything works (maybe because my AirCon is sitting on the same subnet as my Openhab server?).

chutch commented 5 years ago

Well, it's because current version of code doesn't work if you don't provide broadcast address in configuration.

https://github.com/jllcunha/openhab-greeair-binding/blob/97550504d2e24539275a98b4b96b042621f72a6c/org.openhab.binding.greeair/src/main/java/org/openhab/binding/greeair/internal/GreeAirconConfig.java#L50

Though, looking at the logs it seems that both of my devices respond nicely after getting broadcast message, which is a great way for auto discovery.

pezoli commented 5 years ago

The binding is working for me without broadcast ip definition. I can control my 4 Gree AC units, they are sitting on the same subnet as my Openhab server, using the 2.4 version of the binding. Auto discovery function would be nice.

2W817 commented 4 years ago

Works for my cooper & hunter ac perfectly Even without internet (wan disconnected) still can control the ac. Great binding!