home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.94k stars 30.99k forks source link

Ecowitt Water temperature sensor WN34L not showing up in home assistant. #117528

Open godbouy opened 6 months ago

godbouy commented 6 months ago

The problem

I just added this sensor to get my pool temperature. I can get the temperature from the Ecowitt website, but I cannot find the device in Home Assistant. I have tried resetting the device, but it still does not show up. I have added sensors before and they always show up but that is not the case with the WN34L. All other Ecowitt weather station data appears fine with no issues.

What version of Home Assistant Core has the issue?

core-2024.5.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Ecowitt weather station

Link to integration documentation on our website

https://www.home-assistant.io/integrations/ecowitt

Diagnostics information

2024-05-12 20:08:57.438 INFO (MainThread) [custom_components.ecowitt] First ecowitt data recd, setting up sensors. 2024-05-12 20:08:57.438 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 2024-05-12 20:08:57.438 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 2024-05-12 20:08:57.445 WARNING (ImportExecutor_0) [homeassistant.const] DEVICE_CLASS_TIMESTAMP was used from ecowitt, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.TIMESTAMP instead, please report it to the author of the 'ecowitt' custom integration 2024-05-12 20:08:57.453 WARNING (ImportExecutor_0) [homeassistant.const] DEVICE_CLASS_BATTERY was used from ecowitt, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.BATTERY instead, please report it to the author of the 'ecowitt' custom integration 2024-05-12 20:08:57.455 INFO (MainThread) [homeassistant.components.sensor] Setting up ecowitt.sensor 2024-05-12 20:08:57.467 INFO (MainThread) [homeassistant.components.binary_sensor] Setting up ecowitt.binary_sensor 2024-05-12 20:08:58.173 INFO (MainThread) [homeassistant.components.mqtt.client] MQTT client initialized, birth message sent 2024-05-12 20:09:36.080 INFO (MainThread) [custom_components.emporia_vue] Integrating minute data into day sensors 2024-05-12 20:09:36.229 INFO (MainThread) [custom_components.emporia_vue] Could not find usage info for device 87718 channel 5 2024-05-12 20:09:56.548 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 64.4, file a PR. 2024-05-12 20:09:56.548 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:10:10.161 WARNING (Recorder) [homeassistant.components.sensor.recorder] The unit of sensor.openweathermap_rain (mm/h) cannot be converted to the unit of previously compiled statistics (mm). Generation of long term statistics will be suppressed unless the unit changes back to mm or a compatible unit. Go to https://my.home-assistant.io/redirect/developer_statistics to fix this 2024-05-12 20:10:10.161 WARNING (Recorder) [homeassistant.components.sensor.recorder] The unit of sensor.openweathermap_snow (mm/h) cannot be converted to the unit of previously compiled statistics (mm). Generation of long term statistics will be suppressed unless the unit changes back to mm or a compatible unit. Go to https://my.home-assistant.io/redirect/developer_statistics to fix this 2024-05-12 20:10:27.439 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 64.4, file a PR. 2024-05-12 20:10:27.440 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:10:36.079 INFO (MainThread) [custom_components.emporia_vue] Integrating minute data into day sensors 2024-05-12 20:10:36.308 INFO (MainThread) [custom_components.emporia_vue] Could not find usage info for device 87718 channel 5 2024-05-12 20:11:26.544 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 64.4, file a PR. 2024-05-12 20:11:26.545 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:11:36.079 INFO (MainThread) [custom_components.emporia_vue] Integrating minute data into day sensors 2024-05-12 20:11:36.225 INFO (MainThread) [custom_components.emporia_vue] Could not find usage info for device 87718 channel 5 2024-05-12 20:11:57.537 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 64.4, file a PR. 2024-05-12 20:11:57.537 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:12:36.079 INFO (MainThread) [custom_components.emporia_vue] Integrating minute data into day sensors 2024-05-12 20:12:36.261 INFO (MainThread) [custom_components.emporia_vue] Could not find usage info for device 87718 channel 5 2024-05-12 20:12:56.545 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 63.9, file a PR. 2024-05-12 20:12:56.545 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:13:22.996 ERROR (MainThread) [homeassistant.components.hassio.handler] Client error on /addons/a0d7b954_home-panel/changelog request 0, message='Attempt to decode JSON with unexpected mimetype: application/octet-stream', url=URL('http://172.30.32.2/addons/a0d7b954_home-panel/changelog') 2024-05-12 20:13:22.997 WARNING (MainThread) [homeassistant.components.hassio.data] Could not fetch changelog for a0d7b954_home-panel: 2024-05-12 20:13:27.432 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 63.9, file a PR. 2024-05-12 20:13:27.432 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR. 2024-05-12 20:13:32.692 WARNING (MainThread) [aio_georss_client.feed] Fetching data from https://www.gdacs.org/xml/rss.xml failed with 503, message='Service Unavailable', url=URL('https://www.gdacs.org/xml/rss.xml') 2024-05-12 20:13:32.693 WARNING (MainThread) [aio_georss_client.feed_manager] Update not successful, no data received from <GdacsFeed(home=(45.71718378922771, -75.05498690242959), url=https://www.gdacs.org/xml/rss.xml, radius=200, categories=[])> 2024-05-12 20:13:36.080 INFO (MainThread) [custom_components.emporia_vue] Integrating minute data into day sensors 2024-05-12 20:13:36.233 INFO (MainThread) [custom_components.emporia_vue] Could not find usage info for device 87718 channel 5 2024-05-12 20:14:26.555 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_ch1 value 63.9, file a PR. 2024-05-12 20:14:26.555 WARNING (MainThread) [custom_components.ecowitt] Unhandled sensor type tf_batt1 value 1.68, file a PR.

Example YAML snippet

No response

Anything in the logs that might be useful for us?

See diag info. It kind of looks like it could be :
Unhandled sensor type tf_ch1 value 63.9, file a PR.
Unhandled sensor type tf_batt1 value 1.68, file a PR.

Additional information

No response

home-assistant[bot] commented 6 months ago

Hey there @pvizeli, mind taking a look at this issue as it has been labeled with an integration (ecowitt) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `ecowitt` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign ecowitt` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


ecowitt documentation ecowitt source (message by IssueLinks)

stu-pdasso commented 5 months ago

I was thinking the same thing until I notice a soil temp that’s matched what the pool was doing. Ecowitt seems to label them as pool and also soil temp but it seems they come through home assistant as soil temps🤷🏽‍♂️ hopefully that helps.

issue-triage-workflows[bot] commented 2 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

godbouy commented 2 months ago

Everything works fine for me now. I would have loved to share the change but I am not a programmer and modifying the integration on Github is way above my level of expertise.

I modified my ecowitt component in the custom_components folder.

I added these lines in the const.py file and now I get my pool temperature as I wanted.

TYPE_POOL1 = "tf_ch1" TYPE_POOLBAT1 = "tf_batt1"

TYPE_POOL1: ("Pool Temperature 1", DEGREE,
              TYPE_SENSOR, None, "mdi:thermometer", 0),
TYPE_POOLBAT1: ("PoolBattery 1", "BATT", TYPE_BINARY_SENSOR,
              DEVICE_CLASS_BATTERY, "mdi:battery", 0),