-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add support for GW1100A #7
Comments
Thanks @slayer1551 I have a |
|
Thanks @slayer1551 I'll take a look at what's going in the handling with the differences in the data reports. |
Hey @spatialdude I added support for the GW1100A. Right now I have the code in a local branch, I could push it if you want? |
I just received a GW1100B. Hopefully this can get dropped in too. |
Hi @tragnar . Could you show me the mods you made to get the GW1000A working? I am in the same boat. Thanks. |
I am currently not at home till end of month. I can show it to you afterwards. Basically you need to add the GW1100A to every place where the GW1100 is mentioned in the code. And you need to install the plugin from your desktop (this is explained on the GitHub homebridge page somewhere)
Von meinem iPhone gesendet
… Am 12.07.2022 um 18:41 schrieb bc-rbennion ***@***.***>:
Hi @tragnar . Could you show me the mods you made to get the GW1000A working? I am in the same boat. Thanks.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
I have PR #15 open to add support for the GW1100 gateway, and just altered the regex to match either version A or B (also adds support for the WH40 rainfall sensor): |
thanks @tomkinsc ! |
@tomkinsc how do I install your PR on rpi? thanks! |
Ideally @spatialdude will review the PR #15 and merge it or a derived version so the added functionality is available via the 'official' I'm not much of a node person, so I'm not sure how idiomatic this may be, but what I did is more or less:
Of note is that if you upgrade Homekit, the
Important: To anyone considering installing the modified plugin (or anything from a secondary source), I'd encourage reviewing the changes to the code before following the steps above. |
Hi, did @spatialdude already merge the changed code from @tomkinsc, or do I have to make the mentioned steps myself to get the GW1100 working? |
I have also the Gw1100, no error in the log, but the plugin wan´t create any device |
any updates on this. anyone? :) |
Is your feature request related to a problem? Please describe:
No current support for GW1100A
Describe the solution you'd like:
If you can add support for the GW1100A station. The sensors work but it does not show the temperature / humidity / pressure from the base station
Additional context:
I've manually edited the EcowittPlatform.js to add the model GW1100A and it showed up fine in homebridge for me
const modelInfo = dataReport?.model.match(/(HP2551CA|GW1000)_(.*)/);
case 'GW1000': this.baseStationInfo.hardwareRevision = dataReport.stationtype; this.baseStationInfo.firmwareRevision = stationTypeInfo[2]; if (!this.config?.thbin?.hide) { this.addSensorType(true, 'GW1000'); } break;
The text was updated successfully, but these errors were encountered: