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

Test speakers #217

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

joakim-tjernlund
Copy link

No description provided.

Make Mono speker produce proper test adio instead of pink noise
Test Speaker select random output on audio HW with several outputs.
Make it use the the selected ouput instead.
Copy link
Member

@lukefromdc lukefromdc left a comment

Choose a reason for hiding this comment

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

On my setup (HDMI sound from AMD video card over DP available but does not actually work, analog outputs from motherboard work), this had the effect of forcing the tested speakers to always be those chosen in the "output" tab, ignoring the device chosen in the "hardware" tab. This comes from effectively being able to set the chosen device in two different places, and the fact that these two settings can differ.

Might be better if these two were linked together between those tabs in the GUI, so the last device selected in either tab would always be the one actually used-or if the "Hardware" tab were renamed "Test Hardware" and also used only the device selected within itself.

We DO need to unbreak currently broken speaker tests so this might be better than the current defaults but I suspect it as it stands now we'd get confusion and bug reports over it

@joakim-tjernlund
Copy link
Author

The fix is somewhat simplistic but the only thing I can muster. Considering Test Speaker is broken I think this is better
that just leaving this as is.

@lukefromdc
Copy link
Member

OK, I will let the rest of the team decide what to do. We replace a major bug with a lesser one as this stands, I will let the others on the team weigh in on this.

@lukefromdc lukefromdc requested a review from a team March 29, 2024 17:27
@joakim-tjernlund
Copy link
Author

OK, I will let the rest of the team decide what to do. We replace a major bug with a lesser one as this stands, I will let the others on the team weigh in on this.

Well, I was asked to create a PR with these if you look at the issue tracker long time ago. It is a bit late to
have doubts now.

@lukefromdc
Copy link
Member

Not my call to do this solo, so called for other reviews.

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