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

При обновлении НА до 2024.2 перестали обновляться данные #99

Closed
4 tasks done
Rugento opened this issue Feb 9, 2024 · 7 comments · Fixed by #100
Closed
4 tasks done
Assignees
Labels
bug Something isn't working Home Assistant Home Assistant related issues work in progress We a working on it

Comments

@Rugento
Copy link

Rugento commented Feb 9, 2024

В чем проблема?

При обновлении НА до 2024.2 перестали автоматически обновляться данные. В предыдущей версии НА все работало верно, настройки не менялись.
Обновление происходит только при перезагрузке компонента, все данные датчиков правильные.
Не работает только автоматическое обновление. В логах ничего не пишет.

При использовании какой версии интеграции возникает проблема?

3.1.2

Какая версия Home Assistant используется?

2024.2.1

Какой способ установки Home Assistant используется?

Home Assistant Container

Самопроверка по созданию ключа

  • при добавлении ключа в кабинете разработчика он добавлялся как "тестовый"
  • тип ключа в кабинете разработчика был изменен на "погода для сайта"
  • после изменения типа ключа прошло не менее 20 минут
  • в кабинете разработчка я вижу что с запросы делаются именно с этим ключом

Есть что-нибудь полезное в логе?

No response

Дополнительная информация

No response

@tERB00
Copy link

tERB00 commented Feb 10, 2024

То же самое - обновляется теперь только вручную.

@AndreyKurilovich
Copy link

То же самое. Только ручное обновление данных.

@solipse
Copy link

solipse commented Feb 10, 2024

Аналогично, запрос к API не происходит

@IATkachenko IATkachenko added the bug Something isn't working label Feb 10, 2024
@IATkachenko IATkachenko pinned this issue Feb 10, 2024
@IATkachenko IATkachenko self-assigned this Feb 10, 2024
@IATkachenko
Copy link
Owner

IATkachenko commented Feb 10, 2024

Не могу у себя воспроизвести:

2024-02-10 13:31:08.748 DEBUG (MainThread) [custom_components.yandex_weather.weather] Time since last update: 0:06:12.048674 (2024-02-10 10:24:56.699559+00:00), update interval is 0:30:00
...
2024-02-10 13:32:03.256 INFO (MainThread) [custom_components.yandex_weather.updater] Sending API request
2024-02-10 13:32:03.370 DEBUG (MainThread) [custom_components.yandex_weather.updater] {"now":1707561123,"now_dt":"2024-02-10T10:32:03.361059Z","info":{"url":"https://meteum.ai/213?lat=FXFXFXFXF\u0026lon=FXFXFXF","lat":FXFXFXFXF,"lon":FXFXFXFXF},"fact":{"obs_time":1707561123,"temp":-11,"feels_like":-15,"icon":"skc_d","condition":"clear","wind_speed":1.3,"wind_dir":"se","pressure_mm":747,"pressure_pa":995,"humidity":72,"daytime":"d","polar":false,"season":"winter","wind_gust":2.3},"forecast":{"date":"2024-02-10","date_ts":1707512400,"week":6,"sunrise":"08:08","sunset":"17:20","moon_code":8,"moon_text":"moon-code-8","parts":[{"part_name":"evening","temp_min":-17,"temp_avg":-17,"temp_max":-16,"wind_speed":1.6,"wind_gust":4.3,"wind_dir":"e","pressure_mm":749,"pressure_pa":998,"humidity":77,"prec_mm":0,"prec_prob":10,"prec_period":240,"icon":"skc_n","condition":"clear","feels_like":-22,"daytime":"n","polar":false},{"part_name":"night","temp_min":-18,"temp_avg":-17,"temp_max":-16,"wind_speed":2.3,"wind_gust":5.2,"wind_dir":"e","pressure_mm":749,"pressure_pa":998,"humidity":80,"prec_mm":0,"prec_prob":10,"prec_period":480,"icon":"ovc","condition":"overcast","feels_like":-22,"daytime":"n","polar":false}]}}
2024-02-10 13:32:03.383 DEBUG (MainThread) [custom_components.yandex_weather.updater] Finished fetching Yandex Weather updater data in 0.128 seconds (success: True)

Можете debug log показать?
В configuration.yaml нужно добавить

logger:
  default: warning
  logs:
    custom_components.yandex_weather: debug

и перезапустить HA

Для ручного обновления данных можно дернуть службу

service: homeassistant.update_entity
data: {}
target:
  entity_id: weather.yandex_weather

Еще послежу за автообновленияем, может там что-то отвалилось...

@IATkachenko IATkachenko added need more info Further information is requested Home Assistant Home Assistant related issues work in progress We a working on it labels Feb 10, 2024
@IATkachenko
Copy link
Owner

Сломали взаимодействие с координатором в home-assistant/core@8d3f693#diff-c274e5f7f48b4f740b17d6053cfc5019c41576ef1323770ae24ddfb3759561adR230

Сегодня выкачу фикс.

@IATkachenko IATkachenko removed the need more info Further information is requested label Feb 10, 2024
IATkachenko added a commit that referenced this issue Feb 10, 2024
Since home-assistant/core@8d3f693 `update_interval` is a property with setter.
We MUST NOT overload it, because `_update_interval_seconds` was introduced and only new setter define it, but `_schedule_refresh` check its value, instead of checking `update_interval`

Fix #99
@IATkachenko IATkachenko linked a pull request Feb 10, 2024 that will close this issue
IATkachenko added a commit that referenced this issue Feb 10, 2024
Since home-assistant/core@8d3f693 `update_interval` is a property with setter.
We MUST NOT overload it, because `_update_interval_seconds` was introduced and only new setter define it, but `_schedule_refresh` check its value, instead of checking `update_interval`.

Fix #99

* refactor: use modern way to scheduling next update while start
@IATkachenko
Copy link
Owner

Исправлено в v3.1.3.

@shurakr
Copy link

shurakr commented Mar 2, 2024

Стоит 3.1.3 и все равно не работает автоапдайт

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Home Assistant Home Assistant related issues work in progress We a working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants