core icon indicating copy to clipboard operation
core copied to clipboard

Lametric Integration Stopped Working with Sky

Open rbridal opened this issue 1 year ago • 1 comments

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.

rbridal avatar Feb 08 '24 22:02 rbridal

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)

home-assistant[bot] avatar Feb 08 '24 22:02 home-assistant[bot]

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

Joerivanrokeghem avatar Mar 17 '24 08:03 Joerivanrokeghem

Same issue with v3.0.20

bax137 avatar Apr 29 '24 01:04 bax137

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

bax137 avatar Jun 14 '24 19:06 bax137

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

frenck avatar Jun 14 '24 19:06 frenck

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

Joerivanrokeghem avatar Jun 14 '24 19:06 Joerivanrokeghem

Happy to test too, as I have a Sky.

alexh-mathscircle avatar Jun 15 '24 05:06 alexh-mathscircle

Hi @frenck,

Do you got any update for us?

Thanks!

Joerivanrokeghem avatar Jul 10 '24 13:07 Joerivanrokeghem