Changed slider to checkboxes and fixed pulse latency

josh-richardson / cadmus

A GUI frontend for @werman's Pulse Audio real-time noise suppression plugin

GNU General Public License v3.0

2.09k stars 55 forks source link

Changed slider to checkboxes and fixed pulse latency #53

Open pstoebenau opened 3 years ago

pstoebenau commented 3 years ago

I changed the slider to checkboxes because the slider didn't work in some Linux distros.
Pulse audio would have terrible latency when set to 1 millisec. I set it to 200 as a default value that should work for most microphones.

josh-richardson commented 3 years ago

Thanks for this, I'll test it tonight. I might make the checkboxes an option via a parameter or an options menu, rather than the default, as well as making the latency user-configurable.

pstoebenau commented 3 years ago

Yeah it would be awesome if it was user configurable. I do think for now it's best to have defaults though. Because I'm sure this is a problem for a lot of users.

Thanks for your response.

pstoebenau commented 3 years ago

Hey, it's been a while and I think it would be best to just add these changes. The latency I have with the current release is unusable. Thanks again.

IniasP commented 3 years ago

I can confirm the checkboxes work for me whereas the slider doesn't show up (Kubuntu 20.10, Qt 5.14.2).

kristofvloock commented 3 years ago

The checkboxes work for me as well, while the slider doesn't show up (Kubuntu 20.04, Qt 5.12.8).