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

Support reverb with the WASAPI audio driver #836

Closed
jimhen3ry opened this issue Apr 10, 2021 · 5 comments
Closed

Support reverb with the WASAPI audio driver #836

jimhen3ry opened this issue Apr 10, 2021 · 5 comments
Milestone

Comments

@jimhen3ry
Copy link
Contributor

jimhen3ry commented Apr 10, 2021

Related discussion on the mailing list

WASAPI and reverb #831

Is your feature request related to a problem?

FluidSynth reverb setting apparently become ineffective when the Windows WASAPI audio driver is used.

Describe the solution you'd like

Support reverb with the WASAPI audio driver. If exclusive versus non-exclusive WASAPI is an issue, allow a choice between exclusive and non-exclusive modes.

Describe alternatives you've considered

Using the PortAudio WASAPI driver.

Additional context

Reverb is absolutely necessary for virtual organs and WASAPI nearly so. It would be a big help for the Miditzer to be able to allow users to use the WASAPI audio driver without giving up reverb.

@jjceresa
Copy link
Collaborator

Using the PortAudio WASAPI driver.

Have you considered using PortAudio WDM/KS as alternative ?

@jimhen3ry
Copy link
Contributor Author

Have you considered using PortAudio WDM/KS as alternative ?

Yes.

A design goal for the Miditzer is to be very easy for a complete novice to get up and running. We believe that if someone isn't up and running within minutes of downloading, without support or reading documentation, then it is likely they will delete the program and be lost to us. To date, we make dsound the default audio driver because we know it works 99.99% of the time.

When users add a MIDI keyboard the latency of dsound becomes an issue and a lot of support time is spent trying to coach users through tuning their system for reduced latency. We have debated WASAPI versus WDM/KS for many years. The big stumbling block is the lack of understanding of how many users would be negatively affected by using WASAPI or WDM/KS.

We are only now adding support for selecting a FluidSynth audio driver. (The use of FluidSynth is optional in the Miditzer but the vast majority of users do use it.) The addition of WASAPI support in FluidSynth 2.2.0 was serendipitous timing because it coincides with the first major revision of the Miditzer in many years. The Miditzer is strongly driven by the KISS principle. If we can support FluidSynth WASAPI without having to qualify that it kills reverb, that would be our preference. If we can avoid having to talk about adding PortAudio to the mix, we'd like that too.

At this point, nothing is off the table. We are just trying to put together the best "it just works" solution we can.

@jjceresa
Copy link
Collaborator

@jimhen3ry Thanks for the explanations.

If we can avoid having to talk about adding PortAudio to the mix, we'd like that too.

I understand your point. I noticed that portaudio WDM/KS is bugged (using certain audio hardware). So it is best to keep the KISS principle.

@chirs241097
Copy link
Contributor

Should be a fairly easy fix. Will work on it tomorrow.

@derselbst
Copy link
Member

@jimhen3ry I've just merged Chris' PR to master. It will be available in version 2.2.1.

@derselbst derselbst added this to the 2.2 milestone Apr 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants