-
Notifications
You must be signed in to change notification settings - Fork 291
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
Duplicated entities when using home-assistant-matter-hub #2709
Comments
I don’t think we can do anything about it in the hub. I will check why the manufacturer is "TestVendor" instead of "t0bst4r", but that‘s all I can do here… On the other hand, I don’t think there is a clean automated way for alexa-media-player to decide which devices to exclude or include. |
Home Assistant Smart Home Skill (via
TP-Link Skill
Matter Hub
Nightstand Lamp 2:
I managed to spot this is the logs also:
All of the above is in the single, multi-page log entry from the
It will take some analyzing/debugging to track down where the logic is that handles devices added via the official Alexa Integration when using Include devices connected via Echo... |
Thanks @danielbrunt57, good analysis. Let me try to find a way of improving the exclude and include in AMP, it should be something regex compatible or at least a yaml on which we can specify any key and value to include/exclude. As soon as @t0bst4r changes the manufacturer in its code: t0bst4r/home-assistant-matter-hub#245 (comment) We can just filter out by manufacturer and that's it, I think that it will be better to give to users the freedom and choose by their own what to include/exclude other than harcoding it 🤔 |
@danielbrunt57 I thought that I added the link of the code wher this logic is implemented. |
Thanks but I had already looked there straight away and found it.
The comment needs to be corrected as it is inaccurate and very misleading! |
My first thought is:
From there, if we can get the |
FYI, AMP YAML Configuration has/is being deprecated and 5.0.0 now issues a warning to that effect. |
That makes complete senses, I see all devices named like with As a user we can control the name of the hub that it shows as friendlyName, this can lead us to find a way to filter out by hub, but this needs a new logic that is still not in place right? |
Correct! In addition to that, the user will need a way to somehow input that "skill" friendly name exclusion into AMP UI config. |
IMPORTANT: Please search the issues, including closed issues, and the FAQ before opening a new issue. The template is mandatory; failure to use it will result in issue closure.
Describe the bug
I started to use this AddOn that creates a Matter Hub and devices can be controlled locally via Alexa, without using Skills etc:
https://github.com/t0bst4r/home-assistant-matter-hub/
Your application has logic already to handle devices added via the official Alexa Integration when using
Include devices connected via Echo
.I cretead this issue also in here so it can be worked together to find a good solution:
t0bst4r/home-assistant-matter-hub#245
Here an example of a debug logs from a switch device:
Maybe an option to exclude a likst of manufacturers or any other specific key will be useful.
Include devices connected via Echo
Expected behavior
Screenshots
System details
const.py
or HA startup log): latestpip show alexapy
in homeasssistant container or HA startup log): latestDebug Logs (alexa_media & alexapy)
Please provide logs.
Additional context
The text was updated successfully, but these errors were encountered: