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

"Power led dims when audio filtered is disabled" bug #266

Open
johnnovak opened this issue Dec 28, 2023 · 3 comments
Open

"Power led dims when audio filtered is disabled" bug #266

johnnovak opened this issue Dec 28, 2023 · 3 comments

Comments

@johnnovak
Copy link

johnnovak commented Dec 28, 2023

When this option is enabled, it inserts power_led_dim=128 into the config. That indeed causes the power LED to dim if the soft-switchable filter is enabled in the native on-screen display (verified it in Protracker with the E00 and E01 commands). But it also hard-enables the filter on the audio output so it's impossible to turn it off -- this is a bug.

So, the LED toggles alright, but the filter in the audio path is always enabled when this option is on.

Confirmed the behaviour with a brand new clean config created with the Quickstart in 5.1.0 and also in 4.9.1, so it appears this is not a new regression.

@tonioni
Copy link
Owner

tonioni commented Dec 28, 2023

Confirmed and fixed. As a workaround, power_led_dim value less than 96 will not affect filter state.

@johnnovak
Copy link
Author

Thanks Toni.

@johnnovak
Copy link
Author

@tonioni Tested with 5.2.0 RC2, and now the audio is correct regardless of the "Power led dims when audio filtered is disabled" setting. That's good.

But changing the setting at runtime doesn't work, the change is not reflected in the native OSD. You need to save the config and reload it so the state of the option is reflected on the OSD.

Minor thing, but technically it's still buggy, so I thought it's worth mentioning.

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

No branches or pull requests

2 participants