home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
70.75k stars 29.53k forks source link

Lametric Integration Stopped Working with Sky #110058

Open rbridal opened 6 months ago

rbridal commented 6 months ago

The problem

Lametric Sky, a product from Lametric, was working till sometime in the past few months. It is API based like Lametric Time, which does work, but does not implement a few of the features that time does. Notably, sound (which initially caused HA to not be able to work with Sky) and Bluetooth, and has only color screen without monochrome like Time.

Already working integration with Sky stopped working. It looks like the values not presented by Sky are required by the integration - not sure if this was a change to HA or a change to Sky. I tried recreating API keys and removing and re-adding the device to HA. After removing, I cannot add it back in at all, getting errors in the log.

Using manual method to add the Sky device - originally when it was added it was manual, as well.

What version of Home Assistant Core has the issue?

Core-2024.1.6

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

Lametric

Link to integration documentation on our website

https://www.home-assistant.io/integrations/lametric

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.lametric
Source: components/lametric/config_flow.py:142
Integration: LaMetric (documentation, issues)
First occurred: 4:15:55 AM (2 occurrences)
Last logged: 5:02:14 PM

Unexpected error occurred
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/lametric/config_flow.py", line 142, in async_step_manual_entry
    return await self._async_step_create_entry(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/lametric/config_flow.py", line 243, in _async_step_create_entry
    device = await lametric.device()
             ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/demetriek/device.py", line 123, in device
    return Device.parse_obj(response)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic/main.py", line 526, in parse_obj
    return cls(**obj)
           ^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pydantic/main.py", line 341, in __init__
    raise validation_error
pydantic.error_wrappers.ValidationError: 7 validation errors for Device
audio -> volume
  field required (type=value_error.missing)
bluetooth -> name
  field required (type=value_error.missing)
bluetooth -> active
  field required (type=value_error.missing)
bluetooth -> discoverable
  field required (type=value_error.missing)
bluetooth -> pairable
  field required (type=value_error.missing)
bluetooth -> address
  field required (type=value_error.missing)
display -> type
  value is not a valid enumeration member; permitted: 'color', 'grayscale', 'mixed', 'monochrome' (type=type_error.enum; enum_values=[<DisplayType.COLOR: 'color'>, <DisplayType.GRAYSCALE: 'grayscale'>, <DisplayType.MIXED: 'mixed'>, <DisplayType.MONOCHROME: 'monochrome'>])

Additional information

Unfortunately, I can’t add the device in order to get further diagnostics info. Happy to grab anything else needed.

home-assistant[bot] commented 6 months ago

Hey there @robbiet480, @frenck, @bachya, mind taking a look at this issue as it has been labeled with an integration (lametric) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `lametric` 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 lametric` 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)


lametric documentation lametric source (message by IssueLinks)

Joerivanrokeghem commented 5 months ago

I have the same problem, the problem started with SKY firmware version v3.0.20.

bax137 commented 4 months ago

Same issue with v3.0.20

bax137 commented 2 months ago

Still the same issue with sky v3.1.0 and ha 2024.6.2

frenck commented 2 months ago

LaMetric has provided an upstream PR to address this. I'll try to do my best to get it into Home Assistant 2024.7 (and order a Sky myself to test).

../Frenck

Joerivanrokeghem commented 2 months ago

Thanks @frenck for willing to do this. If i can help with testing or something, let me know.

alexh-mathscircle commented 2 months ago

Happy to test too, as I have a Sky.

Joerivanrokeghem commented 1 month ago

Hi @frenck,

Do you got any update for us?

Thanks!