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

Add channel level adjust #338

Open
flikier opened this issue Mar 5, 2025 · 12 comments
Open

Add channel level adjust #338

flikier opened this issue Mar 5, 2025 · 12 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@flikier
Copy link

flikier commented Mar 5, 2025

Marantz Cinema 60 let you change the channel level temporary (does not survive reset).

It is very useful, and I wish to be able to do it through this application.

Here is a screenshot of Marantz application:

Image

Image

@mkulesh
Copy link
Owner

mkulesh commented Mar 9, 2025

Hi @flikier

My Denon receiver does not have such a functionality and I am not able to test it. Furthermore, the documentation related to this Denon/Marantz API is not too precise. In order to implement this feature, I need your support. Could you please collect the application logging as follows:

  • Please manually install MusicControl-v2.14.3-android.apk from here: https://1drv.ms/f/s!At2sh2-YDyGFdsDFl6N3ncDowMg
  • In this app version, navigate to the settings and switch on the "Developer options"
  • Close and open my app again
  • On an another device, start Marantz application as shown on your screenshots. My app should still be running on the first device.
  • In Marantz application (on the second device), try to change all channel levels from your screenshot (Center, Sub, Front L/R, Surr L/R).
  • In my app (on the first device), navigate to application menu -> About -> Latest logging and press "Copy to clipboard"
  • Send this logging to me per email on [email protected]

If you have only one mobile device, please try the desktop version (MusicControl-v2.14.1) and run this version from a desktop (Linux, macOS, or Windows)

@mkulesh mkulesh self-assigned this Mar 9, 2025
@mkulesh mkulesh added the enhancement New feature or request label Mar 9, 2025
@mkulesh mkulesh modified the milestones: Analyzed, In Progress Mar 9, 2025
@mkulesh
Copy link
Owner

mkulesh commented Mar 9, 2025

Developer notice:

  • Onkyo: seems to be not supported by firmware
  • Pioneer: Temporary Channel Level message (TCL)
  • Denon/Marantz: Channel Volume message (CV)

@mkulesh mkulesh changed the title channel level adjust Add channel level adjust Mar 9, 2025
@flikier
Copy link
Author

flikier commented Mar 14, 2025

Thank you very much! 😍

Here is the log:
https://mega.nz/file/n7QjATIA#O-m6tPBfQCKJ5t8kr8GUtLTsP41egN96e-ITwEIJfXc

And here is a screen recording from the second phone that shows what I've done:
https://mega.nz/file/OyZSEIZZ#gVuT28b7MYzUmuMtORqtJyqoVk6AT2yPOOGQqkmGAGk

I wanted to make the screen recording with an exact clock (with seconds), but I've noticed that the log doesn't contain any timestamps... If you still think it would be valuable I could do that.

And I of course would like to help in debugging / testing / whatever needed 🙏🏻

@mkulesh
Copy link
Owner

mkulesh commented Mar 15, 2025

Hi @flikier,

thank you for provided log, it was really helpful. I implemented this feature. Now, you should have a new button "Channel level" in the "Audio Control" window:
Image

When this button pressed, a new window "Channel level" should appear:
Image

In this example, only Subwoofer slider is enabled since my stereo receiver has only this channel. For your receiver, I expect that Front L&R, Center and Surround L&R will additionally enabled.

Please download and manually install MusicControl-v2.14.3 version from here: https://1drv.ms/f/s!At2sh2-YDyGFdsDFl6N3ncDowMg

Could you please install this version and compare MusicControl with the Marantz application, for example change all channel levels in the Marantz application and observe that the levels in MusicControl app are changed accordingly...

@flikier
Copy link
Author

flikier commented Mar 15, 2025

Thank you so much! That was the only feature that kept me using the official unbearable app 😍

Here is a video testing the feature:

https://mega.nz/file/n7QjATIA#O-m6tPBfQCKJ5t8kr8GUtLTsP41egN96e-ITwEIJfXc

In this video there is 1 bug:

  • levels of half decibels are shown as 12 decibels.

And 2 sub-feature requests: 🙏🏻😊

  • support for half decibels.
  • a button to restore all levels to zero.

And in this image - https://mega.nz/file/K2x1Faba#AhkiDlB81ptDBsysYP0ue3LWhjUG-ATI7r3ZtPcqJ38 - I have another 2 suggestions:

  • in red: as it's hard for the eye to count 12 levels, and there is already free space there - maybe show more level indications (4dB, 8dB, etc.)
  • in blue: it is really useful for the main volume to be able to just press + or - for half decibel change, because usually when one changes a level, the change is small. Using the progress bar for such changes, requires a lot of attention in order to do it gently (and it's hard to be accurate). It would be nice also to have such buttons in the new screen.

@mkulesh
Copy link
Owner

mkulesh commented Mar 15, 2025

Hi @flikier

thank you for testing. The first link is broken - instead a video, this link shows the old obsolete log file.

Could you please describe more precisely the bug "levels of half decibels are shown as 12 decibels". May be attach here some screenshots... With respect to the Denon API description, the channel levels are integer in interval [-12dB, 0, +12dB], i.e only 24 integer values without "half decibels"...

I will take a look on other suggestions.

@flikier
Copy link
Author

flikier commented Mar 15, 2025

Sorry, I've pasted the wrong link to the video. Here is the correct one:

https://mega.nz/file/iqYUQKpY#xV-ox-7OSxasM9Oz8lyR9luBsogLkhTYRZM1nc1FkSo

You should see clearly in the video what I mean by having half decibels shown as 12 decibels.

@mkulesh
Copy link
Owner

mkulesh commented Mar 15, 2025

Hi @flikier,
thank you, now I see the problem. The documentation says nothing about "half decibels" therefore I did not consider it. I will fix the problem tomorrow.

@mkulesh
Copy link
Owner

mkulesh commented Mar 16, 2025

Hi @flikier
I hope the bug with "half decibels" is fixed:
Image

Please download and install new build from here: https://1drv.ms/f/s!At2sh2-YDyGFdsDFl6N3ncDowMg

I still working on the remaining proposals...

@flikier
Copy link
Author

flikier commented Mar 16, 2025

Yep, they half decibels bug is fixed 🙏🏻

This improves so much the way I'm enjoying music everyday 😀

@mkulesh
Copy link
Owner

mkulesh commented Mar 23, 2025

Hi @flikier,
I added a new button "Set default" into the channel level dialog:

Image

Please upload the version from here: https://1drv.ms/f/s!At2sh2-YDyGFdsDFl6N3ncDowMg
How does it work for you?

@flikier
Copy link
Author

flikier commented Mar 24, 2025

Yes, "set default" works well. Thanks 😊

Just for the technical curiosity, I'll add that the original application doesn't send a zero-value message for every channel, but it's probably implemented as a single "default" signal that is automatically applied for all channels.
I say that because after clicking "set default" in your application, the behavior screen shows channel by channel with the value zero, but clicking it through the original application doesn't show anything on the AVR screen.

Just sharing it, your implementation works perfectly 🙏🏻

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

No branches or pull requests

2 participants