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

Ecowitt integration (firmware V3.1.5) - 16 soil moisture sensors WH51 #127893

Open
cybercat1970 opened this issue Oct 8, 2024 · 8 comments
Open
Assignees

Comments

@cybercat1970
Copy link

The problem

Ecowitt with firmware 3.1.5 now supports 16 channels for soil moisture sensors (like WH51). Home Assistant Ecowitt integration still shows just 8 of them.

What version of Home Assistant Core has the issue?

core-2024.9.2

What was the last working version of Home Assistant Core?

N/A

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

Ecowitt

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

Firmware 3.1.5 has been released several days ago, the previous versions used 8 channels for soil moisture sensors.

@home-assistant
Copy link

home-assistant bot commented Oct 8, 2024

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)

@cybercat1970
Copy link
Author

Just found in home-assistant.log (sensors 9 and 10 installed):

2024-10-07 17:00:59.652 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 27
2024-10-07 17:00:59.652 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 165
2024-10-07 17:00:59.652 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:02:00.652 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 27
2024-10-07 17:02:00.652 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 165
2024-10-07 17:02:00.653 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:03:01.655 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 26
2024-10-07 17:03:01.656 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 164
2024-10-07 17:03:01.656 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:04:19.764 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 28
2024-10-07 17:04:19.764 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 170
2024-10-07 17:04:19.764 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:05:03.804 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 28
2024-10-07 17:05:03.804 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 170
2024-10-07 17:05:03.804 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:06:04.806 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 28
2024-10-07 17:06:04.806 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 170
2024-10-07 17:06:04.806 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture10 value 15
2024-10-07 17:06:04.806 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad10 value 116
2024-10-07 17:06:04.806 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.7
2024-10-07 17:07:08.768 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 28
2024-10-07 17:07:08.768 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 170
2024-10-07 17:07:08.768 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture10 value 28
2024-10-07 17:07:08.768 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad10 value 163
2024-10-07 17:07:08.769 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt9 value 1.6
2024-10-07 17:07:08.769 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilbatt10 value 1.7
2024-10-07 17:08:06.809 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilmoisture9 value 28
2024-10-07 17:08:06.809 WARNING (MainThread) [aioecowitt.server] Unhandled sensor type soilad9 value 169

@MarkCheshire
Copy link

Looking forward to support for the extra soil sensors. I love this integration so much, I immediately bought extra sensors when the firmware supported it. Just have to be patient now

@zacs
Copy link

zacs commented Oct 23, 2024

I believe this would just be a simple change to the upstream library (aioecowitt). I submitted a PR to handle soil moisture sensors 9-16 here: home-assistant-libs/aioecowitt#209.

@cybercat1970
Copy link
Author

Thanks you, @zacs.
I have appied the fix manually in my Home Assistant instance, everything works fine, additional sensors appeared. No regression noted so far.

@DavidBass77
Copy link

I'm eager to see this working. I've moved all of my sensors onto a single GW2000b for easier management. Just need the integration to give me the higher names.

@stormshaker
Copy link

Same for the latest firmware versions for the other wifi gateways, v2.3.5 on the GW1100c and v1.3.5 on the GW1200C. All support 16 soil sensors now.

@taugen123
Copy link

Thanks @zacs , works nice.

I think calc.py needs also an update on line 190 to ensure int values for the new sensors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants