Help with FDN reverb design | Algorithm Design | Forum

Hi Chris,

 When I load it with the debugger it tells me that it cannot access memory when reading/writing from a circular buffer. 

Can you give the exact output of the debugger? Is this buffer part of your plugin or RackAFX?

Here is the relevant code in various section of the plugin:

Header Variables: 

I assume these are members of your class? Or are they "free"/global variables?

float fdnCoef[24][24] = { 0 }; //Matrix of Coefiecents [current][target]
float globalRC= 0.9; //Global Reflections
float inputMatrix[2][24] = { 0 }; //[inch, node]
float outputMatrix[24][2] = { 0 }; //[node, outch]
double thisInput[24];
double thisOutput[24];
double delayTime[24];
double outputs[2] = { 0 };

ZVAFilter fdnFilters[24]; //Filters at the end of each delay line
SimpleDelay fdnDelay[24]; //Delay lines

You are creating a lot of stuff here, and if the class is instantiated on stack, this might cause a stack overflow.

Also, check how you are accessing memory: You are probably leaving your array bounds here in multiple places:

fdnCoef[n][n + 1] = 1.0;
fdnCoef[n + 1][n + 2] = 1.0;
//To the Left
fdnCoef[n][n + 3] = 0.9;
fdnCoef[n + 1][n + 4] = 0.9;
fdnCoef[n + 2][n + 5] = 0.9;
//To the Right
fdnCoef[n][n - 3] = 0.9;
fdnCoef[n + 1][n - 2] = 0.9;
fdnCoef[n + 2][n - 1] = 0.9;
//Initalize delay lines
fdnDelay[n].reset(resetInfo.sampleRate);
fdnDelay[n].createDelayBuffer(resetInfo.sampleRate, 100.0);

}
for (int n = 0; n < 25; n++) {
delayTime[n] = 10.0;
}

Chris_1 said
Hey Tom,

Thank you for the quick reply.  I have it running now which was due to fixing my array bounds issues (I am a C++ novice and am use to higher level languages just wrapping things for me and there was that one "25" typo). 

You should also take care about the n+1/+2/-2 indices. They go out of bounds at the beginning and the end of your loop, too. C/C++ will allow you to do so - but still you are writing "somewhere" in memory, outside of the array. Probably you are overwriting something else. 

I am doing all of those as free floating variables for now, but I am planning on making the thing into an fdn class that can create a network of n nodes.  Do you think instantiating too many of this class would create stack overflow and is there a better way of dealing with setting the coefficients than passing a huge array?  This is my first foray into lower level programming so I am sure that I am prone to making some rookie mistakes.   

It depends 😉 If you create everything on stack, it will most likely become a problem. You have to consider that arrays in C++ are no separate objects, but they are part of the object's representation in memory (or of "global memory" if you don't use a class). An array of floats as a member variable is (at runtime) same as creating the same number of separate variables:

float data[4];

is equivalent to

float a;
float b;
float c;
float d;

Now, if you create arrays of delays, where each delay has arrays of buffers etc. this will add up really quickly.

The problem is not the amount of memory you need for this, but that you are requesting it as a single block of contiguous memory.

 

Another note: if you place all the variables in "global memory", you won't be able to instantiate more than one plugin!

I hope this helps 😉 If you need more help, maybe we should move the discussion into the "C++ Programming" area.