Skip to content

Add xml files for Marantz M-CR603#16

Merged
ol-iver merged 1 commit into
ol-iver:masterfrom
bmesuere:master
Jun 5, 2017
Merged

Add xml files for Marantz M-CR603#16
ol-iver merged 1 commit into
ol-iver:masterfrom
bmesuere:master

Conversation

@bmesuere
Copy link
Copy Markdown
Contributor

@bmesuere bmesuere commented Jun 4, 2017

When running, the following error is shown:

>>> d = denonavr.DenonAVR("192.168.0.3")
No mapping for audio source TUNER
Missing status information from XML for: FriendlyName

@ol-iver
Copy link
Copy Markdown
Owner

ol-iver commented Jun 5, 2017

I'll fix that in the next version, thank you :)

@ol-iver ol-iver merged commit 4c715bf into ol-iver:master Jun 5, 2017
</RenameSource>
<RenameZone><value></value></RenameZone>
<SourceDelete>
<value>DEL</value>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bmesuere: according to this SourceDelete section you deleted each input source on your receiver. The error message with the missing mapping for the TUNER is raised, because it was deleted.
Please check if you deleted (deactivated) all sources in your web interface or if this is a bug

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only source I deactivated is last.fm. As far as I know, there is no way to disable other sources on the MCR603. I also don't think there is a web interface.

@ol-iver
Copy link
Copy Markdown
Owner

ol-iver commented Jun 7, 2017

Created an option to display the deleted input functions as well
If you instanciate the receiver providing a name and the option, you should not get the error anymore

d = denonavr.DenonAVR("192.168.0.3", "receiver living room", True)
or
d = denonavr.DenonAVR("192.168.0.3", name="receiver living room", show_all_inputs=True)

In homeassistant I created a new option show_all_sources to enable this.
See here: home-assistant/core#7949

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants