Sync of changes

[henrikwidlund]

Hi @ol-iver, as you know I've forked this repo and I've done some modifications. I'm wondering if you want me to contribute some of them to your repo? You can find the diff here: https://github.com/ol-iver/denonavr/compare/main...henrikwidlund:denonavr:main?expand=1

List of things I've added:

• mDNS discovery (my unit and others who are using the UC remote no longer functions with SSDP).
  • This addition also filters on device type so that discovered devices via mDNS and SSDP only contains those who are receivers.
• Rate limiting
  • The receivers usually don't like being spammed with requests which would happen in our case if you held down the volume button for example.
    • The rate limiter is dynamic and takes response times into consideration.
• Custom event for max volume
  • The library used to ignore the MV event when parameter was MAX
  • We started using this because when the max allowed volume is reached the receiver would stop responding when you tried to further increase the volume causing timeouts
• Check current value for a command
  • Sending the same value or a value that is already at min or max causes the receiver to not send a response, timeout again
  • Tests to verify all of this
• Added delay command to set a specific delay
• Added mute toggle command
• Corrected some docs in regards to which values that are accepted
• Changed IMAX filters to use integers instead of strings
• Parallelize some calls made within the setup and update methods
• Added properties for the following:
  • Audio Signal
  • Audio Sound
  • Audio Sampling Rate
  • Video Signal In/Out
  • Input/Output Channels (7.1.4 for example)
  • HDR Input/Output Info
  • Pixel Depth Input/Output
  • Mx FRL Input/Output (mostly seem to be empty, but that is also what the receiver itself shows, maybe the feature isn't completed)
  • Colorspace Input/Output
  • Max Resolution (4K18Gbps for example)
  • The properties above related to video require polling over telnet, they are not propagated reliably even if subscribed to)
• Adjusted the code that handles volume parsing as we noticed that the volume could go out of range (not sure if this was a bug I introduced in my for or not)

[dmutsaers]

I'm looking forward to see these changes added to denonavr.

[ol-iver]

That would be nice @henrikwidlund 🙂
Could you open own PRs for each item? This would be easier to review. I lost track when I just went through all commits 😅

[henrikwidlund]

Okay @ol-iver, I've started the work of adding all of it, starting with the rate limiter. Once that is merged I'll hopefully be able to do a few in parallel without introducing conflicts.