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
73.61k stars 30.77k forks source link

No 5G sensors available #105786

Closed stskrp closed 5 months ago

stskrp commented 11 months ago

The problem

Hello, I have Huawei H122-373 with 5G support, in modem UI I can see information about 5G also, but it is not visible in Home Assistant through integration image image

What version of Home Assistant Core has the issue?

core-2023.12.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Huawei LTE

Link to integration documentation on our website

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

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 11 months ago

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

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


huawei_lte documentation huawei_lte source (message by IssueLinks)

stskrp commented 11 months ago

Also, sensor names looks to be not correct, almost all sensors with same name: H122-373 image

scop commented 10 months ago

Debugging this pretty much requires access to a device that provides those missing attributes, here's a few pointers to get started.

It's possible that support for the 5G things is provided by an endpoint not yet supported by https://github.com/Salamek/huawei-lte-api/. If that is the case, support for them needs to be added there first.

It could also be that we need to pick some things we don't yet pick from the data supplied by huawei-lte-api. If that's the case, it can be addressed in the integration here. To start figuring out if that's the case, I suggest trying the https://github.com/Salamek/huawei-lte-api/blob/master/examples/data_dump.py utility and seeing if the output seems to contain the 5G attributes somewhere. (If posting some of its output to public, be sure to redact any information you don't want to share.)

Regarding the sensor names, that's almost certainly a different thing unrelated to the 5G sensors, please open a separate issue for that.

stskrp commented 10 months ago

Hello, checked data_dump.py, 5G related data is there, but name does nor clearly shows it, 5G related attributes have nr prefix: image

scop commented 9 months ago

So I guess all those that start with nr do get created as sensor entities in HA, they just lack a human readable name there? We don't exclude anything from Device.signal so I think that's what should happen.

Note also that those sensors will likely show up under "Sensors" and not "Diagnostics" in the UI, because we do not have a config for them and thus don't classify them as diagnostics either.

Anyway, thanks for the screenshot with the field names in it, I'll look into adding descriptions for at least some of those.

stskrp commented 7 months ago

Hello, any idea when it can be released/merged?

scop commented 7 months ago

As soon as someone with rights to approve the MR reviews and approves it.