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

Feature Request: fallback when weather sensor fails #27

Open
rigrig opened this issue Jul 7, 2022 · 5 comments · Fixed by #29
Open

Feature Request: fallback when weather sensor fails #27

rigrig opened this issue Jul 7, 2022 · 5 comments · Fixed by #29
Assignees
Labels
enhancement New feature or request

Comments

@rigrig
Copy link

rigrig commented Jul 7, 2022

The last Home Assistant update broke OpenWeatherMap, resulting in the illuminance sensor returning unknown, (log: No attribution attribute: weather.openweathermap.)
My automations expect a numeric value (to prevent flickering the lights when illuminance is unknown during HA restart), so didn't turn on my lights when it got dark.

Maybe instead of bailing out when it can't find/handle current conditions, the illuminance sensor could just fall back to ignoring the weather sensor, so lights at least still turn on/off according to day/night?

@pnbruckner pnbruckner self-assigned this Jul 8, 2022
@pnbruckner pnbruckner added the enhancement New feature or request label Jul 8, 2022
@pnbruckner
Copy link
Owner

Good idea. In fact, I can even make using a weather entity optional, which would just give the curve based solely on sun position.

@luberk8
Copy link

luberk8 commented Jan 5, 2024

Good idea. In fact, I can even make using a weather entity optional, which would just give the curve based solely on sun position.

Did you ever add the weather entity being optional feature? I would like to use your tool "backwards", i.e. calculate "cloudiness" based on an actual sensor

@pnbruckner
Copy link
Owner

@luberk8, no, not yet. In theory the "fallback" feature could do it, but the current implementation expects to see the weather entity at least once, and until it does, the sensor's state will be "unknown". I've been meaning to change that, but I could also explicitly make the weather entity optional. I kind of have that on my "to do" list, just haven't gotten around to it. I'll try to do it sooner rather than later.

@pnbruckner pnbruckner reopened this Jan 5, 2024
@rrooggiieerr
Copy link

give the curve based solely on sun position

I would be very interested in this option but understand this is not yet implemented as weather data source is stil a required argument. Any plans to implement this?

@pnbruckner
Copy link
Owner

Yes, but with all the work I recently did on the Life360 integration, which is now broken, yet again, this kind of got pushed to the "back burner." I'll try to do this soon.

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

Successfully merging a pull request may close this issue.

4 participants