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

4 channels IR files do not work anymore #3

Open
falkTX opened this issue Aug 18, 2013 · 4 comments
Open

4 channels IR files do not work anymore #3

falkTX opened this issue Aug 18, 2013 · 4 comments
Assignees
Labels

Comments

@falkTX
Copy link

falkTX commented Aug 18, 2013

4 channels IR files used to work, but they don't anymore.

I believe either this 03fe806 or this d2c01c9 introduced the regression.

A test case are these free IR files:
http://www.samplicity.com/bricasti-m7-impulse-responses/
The generic wav files work fine, the quad ones do not.

@ghost ghost assigned HiFi-LoFi Aug 26, 2013
@HiFi-LoFi
Copy link
Owner

Hi falkTX,

thanks for reporting!

I just tried to reproduce it on my computer, but here it worked perfectly fine. :-(

In the changes you mentioned, I added additional support for FFTW. Which FFT do you use? The default is OouraFFT which shouldn't have changed. However, if the FFT were wrong, then I'd expect that the convolver doesn't work at all, but you described that the regular stereo case works as expected.

Can you explain a little bit more detailed, what's not working anymore with 4-channel IR files. Does the plugin crash, or isn't there no reverb sound anymore or something like this?

In the non-working 4-channel case, do you know whether already loading of the IRs fails? Are waveforms for all channels displayed in the UI?

Which version of Juce do you use? I'm here still on a little dated version of Juce (2.0.32). Maybe I have to do some adaptions for the current version.

Maybe I should update the KlangFalter to the latest Juce version (maybe this also helps with issue #2).

Thank you so much for your efforts! :-)

Regards, Uli

@falkTX
Copy link
Author

falkTX commented Aug 27, 2013

Hi there.
I'm using a very recent juce tip, just from some days ago. Making your plugin work with latest juce is very welcome, as that will make distrho version diff smaller.

The issue with 4 channels seems to be that the wav files are not read correctly.
and... it seems to be working fine again now, errr.
I'll make more tests and report back.

@HiFi-LoFi
Copy link
Owner

Hi,

I've updated KlangFalter now to the current head of Juce (which is at the moment 42aa27900b49ba75b5a2f15926256307c566c11a). I've also tried to include your changes regarding the settings directory on Linux, so I hope there shouldn't be much diff anymore between this and DISTHRO's version of KlangFalter. :-)

Regarding the 4-channel IRs, anything new here? I also did some testing but everything seems to work fine (but on Mac, because I don't have a Linux system at hand at the moment).

Greetings, Uli

@DoosC
Copy link

DoosC commented Nov 3, 2013

Hello, I just came to have the exact same issue as above with 4-channel IRs.
I'm am running it on Linux also.
The issue is always the same : 2 channels files load just fine but 4 channels are failing to load for some months now.
Unlike falkTX the issue is still here even with the latest updates.
I was about to give up on KlangFalter because the situation has not improved for weeks but he suggested me to report here.
So here I am ;)
It used to work but not anymore (it seems to stop working 2 months ago).
I used the same test files as mentionned above (Wave quad channel files, 32 bit, 48 Khz).
1-1 and 1-2 appear to load fine.
But 2-1 and 2-2 load blank.
The output sound is balanced mostly on one side.
kf1
kf3
kf4
kf2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants