confusing convolution

[A Former User]

@iamlamprey Okay this commit gets rid of the Sample Missing prompt... but I still can't swap IRs in the convolution once I select an expansion

Edit: just noticed my cpu is jumping to 50% now without any FX turned on, I assume because I didn't compile my scriptnode trees (even though they're only a few nodes)

[A Former User]

bumpy bump :) still need help with getting the convolution to NOT use the expansion audiofiles when I load one

[A Former User]

@d-healey said in confusing convolution:

@iamlamprey I did a bit of debugging and found it's looking in the expansion directory

How did you debug it to find out it's looking in the expansion folder? I assume the same thing can work for audiofiles

[d.healey]

@iamlamprey

How did you debug it

Using a debugger (gdb).

[A Former User]

Alright I've tried a bunch of older commits, different expansion folders, different string formats ".wav, .WAV" etc and I still can't get this working.

Once I have an expansion loaded, the convolution tries to load files in the expansion folder instead of the main folder, despite {PROJECT_FOLDER} being present...

inline function onComboBox_AmpCabSelectControl(component, value)
{
    local fullCabName = "{PROJECT_FOLDER}Cab " + Math.round(value) + ".wav";
	Amp_Cab.setFile(fullCabName);
};

Content.getComponent("ComboBox_AmpCabSelect").setControlCallback(onComboBox_AmpCabSelectControl);

This code works on the master branch but not on develop. Idk what changed but it's doing my head in

I thought the ComboBox returning a decimal (1.0) instead of an int was the cause, but it still doesn't work with rounding, and I've tested Console.print() to make sure it was the right string

Out of ideas at this point

[d.healey]

@iamlamprey

Out of ideas at this point

I think this requires a Christoph level fix.

[A Former User]

@d-healey Wish I was a c++ wizard :(

[d.healey]

@iamlamprey You can be, if you start now and work at it for 5+ years :)

[A Former User]

bumpy :)

[A Former User]

Still need help with this... any new Commit I get stuck in a "The Sample directory could not be located" loop.

And any older commit past Master looks for audiofiles in the Expansion folder, despite me using {PROJECT_FOLDER} wild card.

This is the last thing bottlenecking my MacOS release, I haven't changed a line of code since moving from Master to Develop

[d.healey]

@iamlamprey said in confusing convolution:

And any older commit past Master looks for audiofiles in the Expansion folder, despite me using {PROJECT_FOLDER} wild card.

So this issue is fixed now?

[A Former User]

@d-healey Sorry, any newer* commit

Anything on Develop branch is either looking for the audiofiles in the Expansion's subfolders, or its not embedding them in the first place

[A Former User]

I might make a barebones expansion project and see if it can do what I need it to

[d.healey]

@iamlamprey said in confusing convolution:

@d-healey Sorry, any newer* commit

Yeah that sounds similar to the issue I'm having.

[A Former User]

Yep confirmed it with a barebones version:

1. Convolution works fine without an expansion loaded.
2. After installing & loading an expansion, it keeps the IR loaded (still works).
3. If you try to select a new IR, the convolution stops (I assume because it's looking in the wrong folder).
4. If you go back to no expansion loaded, it works again and you can switch files all you want.

@Christoph-Hart could you please take a look at this when you get the chance? I think something's wrong with the {PROJECT_FOLDER} wildcard or something :)

[A Former User]

bump, could we please get some acknowledgement that something's up with expansions? seen several threads about it now and this is the last thing i need working before i can push my mac release

[A Former User]

bump, officially reached 1 month of my convolution not being usable

[Christoph Hart]

It's fixed now, it was indeed looking in the audio files folder of the expansion.

[d.healey]

@christoph-hart Does this solve the includes issue with full expansions?

[Christoph Hart]

@d-healey I don't think so but if you can make a minimal example, I'll take a look (not sure if you did that already).