Skip to content
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

rx-v3900 volume control different #54

Open
konetzed opened this issue Nov 9, 2018 · 3 comments
Open

rx-v3900 volume control different #54

konetzed opened this issue Nov 9, 2018 · 3 comments
Assignees
Labels

Comments

@konetzed
Copy link

konetzed commented Nov 9, 2018

I think the only issue I have found no the RX-V3900 with firmware 1.25 is volume is different

Currently, rxv sends the following

<YAMAHA_AV cmd="PUT"> <Main_Zone> <Volume> <Lvl> <Val>-200</Val> <Exp>1</Exp> <Unit>dB</Unit> </Lvl> </Volume> </Main_Zone> </YAMAHA_AV>

This causes a 400

using mitmproxy I discovered the XML should be

<YAMAHA_AV cmd="PUT"> <Main_Zone> <Vol> <Lvl> <Val>-235</Val> <Exp>1</Exp> <Unit>dB</Unit> </Lvl> </Vol> </Main_Zone> </YAMAHA_AV>

I am wondering how you would like to receive a patch for this? Should it be a new function or should there be an exception for model version?

@pfischbach
Copy link

Trying the same with RX-V3900. Same issue.
konetzed is the SSDP working for you?
receivers = rxv.find() does not work for me.
Any idea?

@konetzed
Copy link
Author

@pfischbach I have two other Yamaha receivers and find worked w/o issue until I put the RX-V3900 on the network. I had previously forgotten to cable it back up after moving it. I have not had time to dig into that issue yet.

I believe it was also mute did not work correctly and needed to be changed from Volume to Vol. Not at the computer with the code on ATM so I am not 100% sure. I am thinking to patch this, for now, I am going to do a check on if receiver version matches 'RX-V3900' the send the Vol commands instead.

Hopefully this weekend I can hack on it some and get a pr in :D

@wuub wuub self-assigned this Jan 19, 2021
@wuub wuub added the bug label Jan 19, 2021
@lbek
Copy link

lbek commented Apr 15, 2021

I would really like to get this bug fixed so I can use home assistant to get and set the volume of my receiver (it's cheaper then buying a new receiver 😬)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants