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
72.18k stars 30.21k forks source link

DPG1C Linak controller not supported by IKEA Idasen Desk integration #104178

Closed datbilling closed 10 months ago

datbilling commented 10 months ago

The problem

Many people including myself have replaced the original Idasen Desk controller with the upgraded Linak DPG1C controller.

There is an issue when using it with the idasen_desk integration though. After a number of hours, moving the desk no longer works, only the current height continues to show. Usually this can be fixed temporarily by disconnecting the controller from HA bluetooth, connecting it to the phone app, moving the desk using the app to "wake it up again" and then disconnecting and reconnecting to HA.

This has recently been fixed in version 2.1.0 of the rhyst/linak-controller project in https://github.com/rhyst/linak-controller/issues/32

Could the same fix be added to the HA idasen_desk integration so that it also supports the DPG1C controller please?

What version of Home Assistant Core has the issue?

2023.11.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

idasen_desk

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 10 months ago

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

Code owner commands Code owners of `idasen_desk` 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 idasen_desk` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


idasen_desk documentation idasen_desk source (message by IssueLinks)

abmantis commented 10 months ago

This would have to be supported by the underlying library: https://github.com/newAM/idasen You can try requesting this feature there. I'm currently not able to look into the required changes, but maybe the library author or someone else would be.

datbilling commented 10 months ago

Ok thanks, I've submitted issue 369 there where I hope someone can help. I would have thought it's promising considering that the newAM/idasen project was orginally forked from the rhyst/linak-controller project where this was already recently fixed.

abmantis commented 10 months ago

@home-assistant close