Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please need more details #1

Closed
rnglad opened this issue Jun 16, 2018 · 69 comments
Closed

Please need more details #1

rnglad opened this issue Jun 16, 2018 · 69 comments
Assignees
Labels
enhancement New feature or request

Comments

@rnglad
Copy link

rnglad commented Jun 16, 2018

Hello,
I have finally found what I was looking for - your plugin, but can’t install it to work.
Can you please describe the process (including mqtt broker) in a bit more detail.
Have 2 gree ACs in home and badly want to control them from home app

@duculete
Copy link
Collaborator

Make sure your AC is connected to your home network via AC. Get it’s ip and configure the plugin. MqTt is needed for getting the room temperature. For example I use DHT 11 sensor with esp8266 that reads room temperature and post it to an mq topic.

@rnglad
Copy link
Author

rnglad commented Jun 16, 2018 via email

@duculete
Copy link
Collaborator

You can use mosquito as mqtt server on a pc or RaspberryPi

@rnglad
Copy link
Author

rnglad commented Jun 16, 2018 via email

@duculete
Copy link
Collaborator

Hello, yes try that and then you should see in logs when it discovers the AC. If any problems please reply here.

@rnglad
Copy link
Author

rnglad commented Jun 18, 2018 via email

@duculete
Copy link
Collaborator

hello,

did you install the plugin?

@rnglad
Copy link
Author

rnglad commented Jun 18, 2018 via email

@duculete
Copy link
Collaborator

I tested on my setup and it running ok.

how did you install the plugin? please post here the accessory configuration for gree-ac

@rnglad
Copy link
Author

rnglad commented Jun 19, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jun 19, 2018 via email

@duculete
Copy link
Collaborator

I see. I will take a look since I have only one AC and I did not tested with multiple AC on the same network.

@rnglad
Copy link
Author

rnglad commented Jun 19, 2018 via email

@duculete
Copy link
Collaborator

I uploaded to github the changes. you can test by getting the contents of app/deviceFactory.js (from this repo) update the file /usr/local/lib/node_modules/homebridge-gree-ac/app/deviceFactory.js (replace the contents of the file your raspberry pi with the code from this repo); do the same thing for index.js (from this repo) -> /usr/local/lib/node_modules/homebridge-gree-ac/index.js

After that, restart homebridge and you should see all 3 AC added to Home app and min temperature 16C
... or you can download the entire repo, install with npm and test.
let me know if you have troubles and I can help.

@duculete duculete self-assigned this Jun 19, 2018
@duculete duculete added the enhancement New feature or request label Jun 19, 2018
@rnglad
Copy link
Author

rnglad commented Jun 19, 2018 via email

@duculete
Copy link
Collaborator

that is an invalid config. GreeAC is the name of the accessory (this is telling Homebridge witch plugin to use), so it should be the same for all the "instances". value of "name" field will appear also on Home app on your iPhone (but you can can rename it on your Iphone, later).

The only thing that differentiate accessories is the "name" key". Use this configuration:

"accessories": [
{
"accessory": "GreeAC",
"host": "192.168.1.136",
"name": "Office AC",
"debug": "false",
"updateInterval": 10000,
"mqttUrl": "mqtt://127.0.0.1",
"currentTempTopic": "home/office/temperature"
},
{
"accessory": "GreeAC",
"host": "192.168.1.54",
"name": "BedRoom AC",
"debug": "false",
"updateInterval": 10000,
"mqttUrl": "mqtt://127.0.0.1",
"currentTempTopic": "home/bedroom/temperature"
},
{
"accessory": "GreeAC",
"host": "192.168.1.253",
"name": "LivingRoom AC",
"debug": "false",
"updateInterval": 10000,
"mqttUrl": "mqtt://127.0.0.1",
"currentTempTopic": "home/LivingRoom/temperature"
}

@rnglad
Copy link
Author

rnglad commented Jun 19, 2018 via email

@duculete
Copy link
Collaborator

Hello,

Can you try with one accessory defined? First configure ip of the first AC, restart Homebridge and test. Next stop homebridge, and test for the ip of the second AC. Same for the last AC device.

Thanks,

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@duculete
Copy link
Collaborator

Give it a few seconts, it will update the home app also.

@duculete
Copy link
Collaborator

Ok, i did a test by adding a "fake" AC but on the same IP. So now I have 2 AC in home app (but both will controll my home AC). It works ok for me, no crash. Can you test? Just take the /app/deviceFactory.js from this repo and replace your /app/deviceFactory.js (on your Mac or where you run Homebridge), put the configuration with 3 AC (valid ips), restart homebridge and try again.

thanks,

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@duculete
Copy link
Collaborator

hmm, how about now?

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jun 20, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

you mentioned that the errors remained - what are the errors?
What is the status of Home app accessories?

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

are you sure you are using the last code (deviceFactory.js)? i don't see any messages like:

init deviceFactory on host ...

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

You get the error on home app on iphone? Try to stop homebridge, remove persistent and accesories folders from homebridgehome folder and restart homebridge.

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

Hello,

I uploaded the code that works for me (from my homebridge instance). Can you test now?

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

what icon? can you be more specific?

@duculete
Copy link
Collaborator

duculete commented Jul 9, 2018

img_3294
img_3295

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 9, 2018 via email

@duculete
Copy link
Collaborator

Did you started? Long press on icon/mode/cool.
Do you have mqtt topic with room temperature ?

@duculete
Copy link
Collaborator

duculete commented Jul 10, 2018

Are you sure that your AC type is exposing same api as Gree AC? Please post here a screenshot of your home app.

@rnglad
Copy link
Author

rnglad commented Jul 10, 2018 via email

@duculete
Copy link
Collaborator

Cool! Enjoy!

@rnglad
Copy link
Author

rnglad commented Jul 10, 2018 via email

@duculete
Copy link
Collaborator

Hello,
I will update the code on npm after few days of (you) testing :D.

What is your configuration in homebridge and how does your temperature sensor write data to mqtt topic?

@rnglad
Copy link
Author

rnglad commented Jul 11, 2018 via email

@duculete
Copy link
Collaborator

You need to read again the description of the plugin. My plugin reads room temperature from mqtt topic...

@rnglad
Copy link
Author

rnglad commented Jul 11, 2018 via email

@duculete
Copy link
Collaborator

I explained it in my first comment 25 days ago. Look at the begining of this discussion...

@rnglad
Copy link
Author

rnglad commented Jul 29, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 31, 2018 via email

@rnglad
Copy link
Author

rnglad commented Jul 31, 2018 via email

@duculete
Copy link
Collaborator

Thanks for the update! Enjoy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants