-
-
Notifications
You must be signed in to change notification settings - Fork 31.5k
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 - the sensor ********_wind_direction does not have a 'state_class' attribute #129260
Comments
Hey there @pvizeli, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) ecowitt documentation |
I have the same issues, even with more entities. I am on an older version of Ecowiitt, perhaps that is the problem but the newer one broke the connection (SSL issues i think), so I went back. Now HA gives me all kind of repair warnings. |
@Janthenman67, if we are talking about the "official" Ecowitt integration (and not the one that is available through HACS), then I have found a "temporary" solution for this integration (until the developer finds time to make corrections) ... |
Pls share it! |
@dwd1, hello!
ATTENTION!!! When restarting the Home Assistant, the attribute does not disappear. Well, that's about it... I hope this can be useful to someone... |
Thank you!
|
@pvizeli can u add this 1 line to the code? |
@dwd1, hello!
Yes, it can probably be done just as easily in the case of Home Assistant Supervised too... It's just that I have very minimal knowledge in docker and other similar things... |
@tkunzfeld, thank you! |
Yes I think so. This is my first PR in a project like this and at first I did something wrong with CLA, but I think this should be fixed now and all that is left is the review of the code owner. |
Add state_class to EcoWittSensorTypes.DEGREE (#129260)
Yes, I've been watching what's going on... Thanks! |
The problem
Good afternoon.
Basically, I outlined the problem in the title: the sensor
********_wind_direction
created by Ecowitt integration does not have astate_class: measurement
attribute .Therefore, the values of this sensor cannot be used correctly, for example, with this custom card: https://github.com/aukedejong/lovelace-windrose-card
When specifying the
********_wind_direction
sensor in this card forwind_direction_entity
, the card is not displayed correctly...Please add this attribute.
What version of Home Assistant Core has the issue?
core-2024.10.4
What was the last working version of Home Assistant Core?
No response
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
I understand very little about this, but it seems to me that to solve this problem it is necessary to add a line
state_class=SensorStateClass.MEASUREMENT
for theEcoWittSensorTypes.DEGREE
in theECOWITT_SENSORS_MAPPING
section of thesensor.py
file...If you don't have time to make changes (perhaps much deeper changes are needed than those I suggested), I ask you to indicate what needs to be changed and how - perhaps I can do it myself in my HA...
The text was updated successfully, but these errors were encountered: