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

Naming of the WH32 sensor and others in the plugin configuration #4

Open
mbrowny opened this issue May 17, 2021 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@mbrowny
Copy link

mbrowny commented May 17, 2021

Is your feature request related to a problem? Please describe:

In the plugin configuration the WH32 sensor is called: Indoor Thermometer/Hygrometer/Barometer Sensor
But on the ecowitt website: https://www.ecowitt.com/shop/goodsDetail/52 it is described as: "Outdoor Temperature and Humidity Sensor (WH32)"

Also include the sensor types in the other descriptions.

Describe the solution you'd like:
Suggest to rename the "Indoor Thermometer/Hygrometer/Barometer Sensor" to "Outdoor Temperature/Humidity Sensor" and add the sensor type in a bracket.

Additional context:
Would be easier for someone installing the plugin to understand which sensor it actually is.

@mbrowny mbrowny added the enhancement New feature or request label May 17, 2021
@spatialdude
Copy link
Owner

@mbrowny The problem I had when developing the plugin was that the data payload doesn't always clearly detail the types of sensors the data is coming from as you can see in the example in #2 (comment)

In my case I have two installations. A GW1000 connected to an older style WH24 like this one
image
and HP2551 with the newer outdoor sensors like this one WS
image, both with additional other sensors.

I have a range of, but not all of, the sensors that Ecowitt sells so have tried to figure out how to interpret the data payload relationship to sensors based on available information.

I remember at the time being a little confused about the WH32 and the details on the website as you also point out. The indoor sensor that came with this system https://www.ecowitt.com/shop/goodsDetail/1 is a WH32 (I need to confirm this but am away from the installation to be sure).

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