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.41k stars 30.3k forks source link

Errors log file related to Denon AVR #73680

Closed meierthomas closed 1 year ago

meierthomas commented 2 years ago

The problem

Starting a new Home Assistant to test the Denon AVR integration for a Denon AVR-3313 and Denon AVR-1713.

I get this error "some attributes not found on update". The receivers seem to ok work right in Home Assistant but I would love to fix this and understand what's causing it.

2022-06-14 02:10:08 WARNING (MainThread) [denonavr.foundation] Error verifying Appcommand.xml update method, deactivate this interface 2022-06-14 02:10:08 ERROR (MainThread) [homeassistant.components.denonavr.media_player] Error Some attributes of zone Main not found on update: {'_power': './Power/value'} occurred in method async_update for Denon AVR receiver Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/components/denonavr/media_player.py", line 181, in wrapper return await func(self, *args, **kwargs) File "/usr/src/homeassistant/homeassistant/components/denonavr/media_player.py", line 232, in async_update await self._receiver.async_update() File "/usr/local/lib/python3.9/site-packages/denonavr/denonavr.py", line 170, in async_update await self._device.async_update(global_update=True, cache_id=cache_id) File "/usr/local/lib/python3.9/site-packages/denonavr/foundation.py", line 127, in async_update await self.async_update_power( File "/usr/local/lib/python3.9/site-packages/denonavr/foundation.py", line 359, in async_update_power await self.async_update_power_status_xml(cache_id=cache_id) File "/usr/local/lib/python3.9/site-packages/denonavr/foundation.py", line 455, in async_update_power_status_xml raise AvrProcessingError( denonavr.exceptions.AvrProcessingError: Some attributes of zone Main not found on update: {'_power': './Power/value'} 2022-06-14 02:10:18 ERROR (MainThread) [homeassistant.components.denonavr.media_player] Error Some attributes of zone Main not found on update: {'_power': './Power/value'} occurred in method async_update for Denon AVR receiver Traceback (most recent call last):

What version of Home Assistant Core has the issue?

core-2022.6.0

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

Denon AVR

Link to integration documentation on our website

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

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

probot-home-assistant[bot] commented 2 years ago

denonavr documentation denonavr source (message by IssueLinks)

probot-home-assistant[bot] commented 2 years ago

Hey there @ol-iver, @starkillerog, mind taking a look at this issue as it has been labeled with an integration (denonavr) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

meierthomas commented 2 years ago

Here's the log with some additional debug information

Denon AVR errors.log

Anto79-ops commented 2 years ago

juist wanted to confirm I get similar errors on start up usaully. When I check the AVR receiver in question, state is unknown. I releod the integration and its fine.

could this be solved with this fix coming in the next update?:

https://github.com/home-assistant/core/pull/73806

meierthomas commented 2 years ago

I have run the python script as advised by @ol-iver in issue #70176 and here's the output. I have two Denon receivers; an AVR-3313 and an AVR-1713. Not sure this is of any help but I hope there's a resolution to this other than to uninstall the integration :)

denonavr_3313_1713.zip

meierthomas commented 2 years ago

Hi @ol-iver, @starkillerOG do you have any idea what the issue could be? It would be great to get an understanding whether this is something known (and is worked on or not). Given I'm not a developer I have no means to progress this. txs

starkillerOG commented 2 years ago

@ol-iver I don't quite understand what is going on, the power value does seem to be present in the uploaded xml files.

zuloomi commented 2 years ago

Please let me know if there's any other diagnostic info I can provide. Both receivers are on static IPs

hashtag8 commented 2 years ago

@ol-iver I'm having a similar "Main not found on update: {'_power': './Power/value'} occurred in method async_update for Denon AVR receiver" error on a 4310CI. However in my case zone 2/3 power values are not present. Appending the zone name (goform/formMainZone_MainZoneXml.xml?ZoneName=Zone2) produces an XML with the power value. Also seems to go into an endless loop attempting to discover the xml values so I have the integration disabled for now...can provide additional info if needed.

denon 4310ci
github-actions[bot] commented 1 year ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.