You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
And the next line tries to check if there were actual data. But the problem is, if the original data source never existed, this conversation will fail, and cannot continue. For exmaple, with openweathermap provider for my location, the wind gust speed data is not available, but generally handling this case for other parts of the code should be updated. One possible solution should be to add a default None value here, like: {% set wind_gust_speed = state.attributes.get('wind_gust_speed') | float(None) %}
At least I guess this should do the trick, but didn't try.
The text was updated successfully, but these errors were encountered:
I'm back from vacation, and I'll take a look at the issue. Interestingly, when I tested it, there were no errors :( It's possible something has changed.
But the code is getting so complicated that I might just revert it to its original state.
if all the data exist, then no problem will occur :)
I don't think any revert is needed, this looks kind of small fix, I just don't have a development setup for this. But as far as I can tell, a default None value for the float conversions should be enough
v1k70rk4
added a commit
to v1k70rk4/intents
that referenced
this issue
Jan 14, 2025
https://github.com/home-assistant/intents/blob/main/responses/hu/HassGetWeather.yaml
There are multiple conversation to float like this: https://github.com/home-assistant/intents/blob/main/responses/hu/HassGetWeather.yaml#L178
And the next line tries to check if there were actual data. But the problem is, if the original data source never existed, this conversation will fail, and cannot continue. For exmaple, with openweathermap provider for my location, the wind gust speed data is not available, but generally handling this case for other parts of the code should be updated. One possible solution should be to add a default None value here, like:
{% set wind_gust_speed = state.attributes.get('wind_gust_speed') | float(None) %}
At least I guess this should do the trick, but didn't try.
The text was updated successfully, but these errors were encountered: