-
-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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
WeatherFlow Cloud - Weatherflow REST - Failed Setup, will retry 'precip_icon' #112841
Comments
Hey there @jeeftor, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) weatherflow_cloud documentation |
I am experiencing the same issue. (No issues with the local Tempest integration.) |
If either of these PRs gets merged should fix the issue I believe |
Same issue can load de integration, I attach the debug file |
I loaded 2024.3.1 and see reference to bump of weatherflowpy and you mention the merge above, yet issues persists. |
Ugh. I’ll look first thing in the morningSent from my iPhoneOn Mar 9, 2024, at 1:32 PM, home-assistant[bot] ***@***.***> wrote:
Hey there @jeeftor, mind taking a look at this issue as it has been labeled with an integration (weatherflow_cloud) you are listed as a code owner for? Thanks!
Code owner commands
Code owners of weatherflow_cloud 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 weatherflow_cloud 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)
weatherflow_cloud documentation
weatherflow_cloud source
(message by IssueLinks)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
So now I'm confused ... In my current version of the lib there is a test to validate that field is optional and it passes so I'm wondering if something didn't make it into the point release yet. Looks like you should have |
May take a version or two more to get this right... hoping to get better debug info into the backing lib as well. |
Thanks, appreciate it! Keep up the awesome work. As much as I love running things locally, I prioritize security when feasible and prefer to keep the weatherflow on an isolated VLAN with all my IoTs so this integration is game changer to finally have functional localized weather, even if I have to hit the cloud for it. |
Are you able to modify the manifest to version |
I just refactored the backing library test infrastructure so moving forward if I can get "bad" data from people I can just toss it in and run the tests against everything in there |
Or maybe can you turn on |
Not sure I've ever done that but if you can tell me what you need done, I can do it, else your last comment regarding debug in config.yaml, I can surely do that. |
Here is the debug too. KeyError: 'precip_icon' |
So I'm about 90% sure this will be solved once we get a release with the correct backing lib version. In the short term if you are motivated you can pull your data from here:
And I can look at it ... but otherwise I really do "think" we are in the clear currently |
Has correct lib version been implemented? On 2024.4.0 and issue still persists. |
Can you post the offending data. Have t upgraded yet Sent from my iPhoneOn Apr 3, 2024, at 1:19 PM, Chris ***@***.***> wrote:
Has correct lib version been implemented? On 2024.4.0 and issue still persists.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I need your test data I guess to validate everything ... and we can track this down because I'm "done gone confused" (probably because in colorado I only get a limited set of weather conditions) |
I'm in Florida so def see different weather than you. Let me know what you need and I'll grab it for you. |
Here are the three that keep showing up alot including the precip_icon. precip_icon Unexpected error fetching weatherflow_cloud data: 'precip_icon' Unable to convert Data - See Attachment Logger: homeassistant.components.weatherflow_cloud Unexpected error fetching weatherflow_cloud data: 'Very Heavy Rain' is not a valid Condition Unexpected error fetching weatherflow_cloud data: 'Very Heavy Rain' is not a valid Condition |
So I'll push a patch with heavy ran and I'm going to make |
Actually where can I send the logs direct to you? |
jeef at vookie.net I think
…On Thu, Apr 4, 2024 at 9:13 AM Chris ***@***.***> wrote:
Actually where can I send the logs direct to you?
—
Reply to this email directly, view it on GitHub
<#112841 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABRQ4X25UTMTSMOPGCPPZ3TY3VUYNAVCNFSM6AAAAABEOLW6JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZXGQ4DGOBQGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
or on discord
…On Thu, Apr 4, 2024 at 9:26 AM Jeff Stein ***@***.***> wrote:
jeef at vookie.net I think
On Thu, Apr 4, 2024 at 9:13 AM Chris ***@***.***> wrote:
> Actually where can I send the logs direct to you?
>
> —
> Reply to this email directly, view it on GitHub
> <#112841 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABRQ4X25UTMTSMOPGCPPZ3TY3VUYNAVCNFSM6AAAAABEOLW6JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZXGQ4DGOBQGE>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
logs sent via email let me know where to grab the JSON data you need and I will send that over too. |
Here is the offending data: {
"air_temp_high": 22.0,
"air_temp_low": 17.0,
"conditions": "Clear",
"day_num": 6,
"day_start_local": 1712376000,
"icon": "clear-day",
"month_num": 4,
"precip_probability": 0,
"sunrise": 1712402105,
"sunset": 1712447494
}, |
and PR submitted.. :) |
Looks like the fix made it into 2024.4.1. So far all is working... Will monitor in the coming days for any issues. |
All hail the finally being fixed!!!! |
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. |
The problem
Attempting to setup the newer option for WeatherFlow Cloud
After entering personal API Key see the following
WeatherFlow Cloud - Weatherflow REST - Failed Setup, will retry 'precip_icon'
What version of Home Assistant Core has the issue?
core-2024.3.0
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
WeatherFlow Cloud
Link to integration documentation on our website
https://www.home-assistant.io/integrations/weatherflow_cloud
Diagnostics information
Logger: homeassistant.components.weatherflow_cloud
Source: helpers/update_coordinator.py:318
integration: WeatherflowCloud (documentation, issues)
First occurred: 2:56:49 PM (35 occurrences)
Last logged: 3:30:52 PM
Unexpected error fetching weatherflow_cloud data: 'precip_icon'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 318, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/weatherflow_cloud/coordinator.py", line 35, in _async_update_data
return await self.weather_api.get_all_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/weatherflow4py/api.py", line 137, in get_all_data
weather=await self.async_get_forecast(station_id=station.station_id),
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/weatherflow4py/api.py", line 85, in async_get_forecast
return await self._make_request(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/weatherflow4py/api.py", line 44, in _make_request
response_model.from_json(data)
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/api.py", line 63, in from_json
return cls.from_dict(kvs, infer_missing=infer_missing)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/api.py", line 70, in from_dict
return _decode_dataclass(cls, kvs, infer_missing)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 216, in _decode_dataclass
value = _decode_dataclass(field_type, field_value,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 220, in _decode_dataclass
init_kwargs[field.name] = _decode_generic(field_type,
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 291, in _decode_generic
xs = _decode_items(get_type_arg_param(type, 0), value, infer_missing)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 399, in _decode_items
return list(_decode_item(type_args, x) for x in xs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 399, in
return list(_decode_item(type_args, x) for x in xs)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 372, in _decode_item
return _decode_dataclass(type_arg, x, infer_missing)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/dataclasses_json/core.py", line 172, in _decode_dataclass
field_value = kvs[field.name]
~~~^^^^^^^^^^^^
KeyError: 'precip_icon'
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: