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

FEAT(client): Introduce Positional Audio Viewer, replacement for MumblePAHelper #5127

Merged

Conversation

davidebeatrici
Copy link
Member

@davidebeatrici davidebeatrici commented Jun 16, 2021

MumblePAHelper (Mumble Positional Audio Helper) is the tool we had been using to test positional audio plugins.

As the application is basically just a dialog, we thought many times in the past about merging it in Mumble.

Now that MumblePAHelper is obsolete (it doesn't support the new API), it's the perfect time to do that.

The dialog can be opened through a new action in the "Developer" menu.

Showcase: https://odysee.com/positional-audio-viewer-left4dead:f4ad40f7e55797e6c8d2b1817915cab71b4893f4

@davidebeatrici
Copy link
Member Author

I hate that PositionalAudioViewer.ui's content is in random order...

src/mumble/CMakeLists.txt Outdated Show resolved Hide resolved
src/mumble/MainWindow.ui Outdated Show resolved Hide resolved
src/mumble/dialogs/PositionalAudioViewer.cpp Outdated Show resolved Hide resolved
src/mumble/dialogs/PositionalAudioViewer.cpp Outdated Show resolved Hide resolved
src/mumble/dialogs/PositionalAudioViewer.cpp Outdated Show resolved Hide resolved
src/mumble/dialogs/PositionalAudioViewer.h Outdated Show resolved Hide resolved
@Krzmbrzl Krzmbrzl added the feature-request This issue or PR deals with a new feature label Jun 16, 2021
@davidebeatrici davidebeatrici force-pushed the positional-audio-viewer branch from 7e47c86 to 8dcf12e Compare June 16, 2021 21:42
src/mumble/PositionalAudioViewer.cpp Outdated Show resolved Hide resolved
src/mumble/PositionalAudioViewer.cpp Show resolved Hide resolved
src/mumble/PositionalAudioViewer.cpp Show resolved Hide resolved
@davidebeatrici davidebeatrici force-pushed the positional-audio-viewer branch from 8dcf12e to db96322 Compare June 18, 2021 08:46
…lePAHelper

MumblePAHelper (Mumble Positional Audio Helper) is the tool we had been using to test positional audio plugins.

As the application is basically just a dialog, we thought many times in the past about merging it in Mumble.

Now that MumblePAHelper is obsolete (it doesn't support the new API), it's the perfect time to do that.

The dialog can be opened through a new action in the "Developer" menu.

Showcase: https://odysee.com/positional-audio-viewer-left4dead:f4ad40f7e55797e6c8d2b1817915cab71b4893f4
@davidebeatrici davidebeatrici force-pushed the positional-audio-viewer branch from db96322 to 0a2c555 Compare June 22, 2021 22:27
@davidebeatrici davidebeatrici requested a review from Krzmbrzl June 23, 2021 05:32
@davidebeatrici davidebeatrici merged commit 12abc4a into mumble-voip:master Jun 23, 2021
@davidebeatrici davidebeatrici deleted the positional-audio-viewer branch June 23, 2021 06:58
@vimpostor
Copy link
Contributor

vimpostor commented Jun 23, 2021

@davidebeatrici
I think it would be a good idea to add a note to the readme of the mumble-pahelper repo and then to mark that entire repo as archived, so that noone will be confused why the separate mumble-pahelper does not work anymore.

@Krzmbrzl
Copy link
Member

@vimpostor see mumble-voip/mumble-pahelper#25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client feature-request This issue or PR deals with a new feature positional audio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants