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
71.09k stars 29.75k forks source link

BMW Connected Drive Error "Unable to update tires" #114742

Open stalakerob opened 5 months ago

stalakerob commented 5 months ago

The problem

Connected Drive logs error: Unable to update tires - (TypeError) TireState.init() missing 1 required positional argument: 'status'

What version of Home Assistant Core has the issue?

core-2024.3.3

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

BMW Connected Drive

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: bimmer_connected.vehicle.vehicle
Quelle: components/bmw_connected_drive/coordinator.py:57
Erstmals aufgetreten: 11:21:46 (30 Vorkommnisse)
Zuletzt protokolliert: 13:47:02

Unable to update tires - (TypeError) TireState.__init__() missing 1 required positional argument: 'status'

Additional information

No response

home-assistant[bot] commented 5 months ago

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

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


bmw_connected_drive documentation bmw_connected_drive source (message by IssueLinks)

rikroe commented 5 months ago

Please provide the diagnostics (either from the integration or the device). This would help a lot!

stalakerob commented 5 months ago

config_entry-bmw_connected_drive-da26af883b3b5a69691f4e0432b55735.json

rikroe commented 5 months ago

Huh - interesting. It seems that the data is returned by the API and was parsed correctly. Does this happen regularly or was it just today?

stalakerob commented 5 months ago

This happened only once so far after changing winter to summer tires.

rikroe commented 5 months ago

OK. I'll have a look if it is possible to gracefully get around this while no tires are on the vehicle (however this is managed on BMW side).

stalakerob commented 5 months ago

Don't worry. Since this only happens during tire change its not a big deal.

sincze commented 2 months ago

Would it be correct to assume whilst reading the above and https://github.com/bimmerconnected/bimmer_connected/discussions/483 info regarding tires (including pressure) and lids are available in HA ?

The Debug log of HA shows the data, but I do not have the sensor.

            "frontRight": {
              "details": {
                "identificationInProgress": false
              },
              "status": {
                "currentPressure": 260,
                "targetPressure": 240
              }
            },
rikroe commented 2 months ago

No, tire details are not yet available in HA. It requires some changes how values are selected from the backend library.

The error exists as tire data is still parsed completely in the backend.

RLProteus commented 1 month ago

I can confirm this is now working for me on version 2024.8.0b1. Thanks!

image

sincze commented 1 month ago

I am new to HA so how can I add this version to HA to see what it does ?? ;-)

rikroe commented 1 month ago

If you're new to HA, best wait until tomorrow and then upgrade to the next HA version (2024.8). Currently it is only available in the beta.

sincze commented 1 month ago

If you're new to HA, best wait until tomorrow and then upgrade to the next HA version (2024.8). Currently it is only available in the beta.

Ok. I can wait.

But you know I'd like to help where possible :) In Domoticz I am quite familiar. HA different ball game.