confusing convolution

[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...

https://youtu.be/bnNzuwgKgwg

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).

[d.healey]

@christoph-hart said in confusing convolution:

not sure if you did that already

Me neither :) but I shall whip one up very soon.

[d.healey]

@Christoph-Hart Here's the full expansion project and also the test expansion with the included scripts. I notice that the scripts are inlined when the expansion is loaded. I'm not sure if the issue is caused when the hxi is generated or when the expansion is loaded.

I also noticed the using Engine.loadFontAs("{PROJECT_FOLDER}FontName.ttf", "pretty-name"); causes issue too because once the expansion loads it's still looking in the PROJECT_FOLDER instead of the expansion folder. Even if I include the font in the shell's images folder it won't work. I assume the best way to handle this would be to have the hxi exporter replace PROJECT_FOLDER with EXP::EXPNAME ? That way it would work while developing the expansion but also work when loaded as an expansion...

I haven't tested but I think this would affect anything that's using PROJECT_FOLDER paths, like additional source code, sample maps, and images.

Test Exp1.zip
Full Expansion Test.zip

[Christoph Hart]

@d-healey Oh wow, that might be the commit with the weirdest looking fix in the history of HISE...

https://github.com/christophhart/HISE/commit/2bdd95a60d094c577dd7a0199130756c7de955c3

Like you see the code change and immediately know, "yeah, that makes total sense".