ProffieOS 7.7 Beta (done)

The fix has been merged on github.

I think once this PR is done:

We can maybe move from Apha to Beta stage.
Once we go to beta, I will create a 7.x branch on github, which means that the default for any new feature added will be for it to go into 8.x. Adding more features to 7.x will need a reason for it.

I beleive I’ve addressed all the open conversations over there on that PR.

Additionally, does the SoundFonts overhaul project matter as far as coinciding with OS7?
https://github.com/profezzorn/SoundFonts/pulls

1 Like

No, that is a separate project.

Although, we may want to change the default_proffieos_config to only use fonts that exists in both the old and the updated default sd card download. (Or maybe use search paths, like “oldfont;newfont”.

Taking this to here:

1 Like

yay, this was needed!!! The stock fonts are all awesome, but all needed some updating! Thanks BC!

Ok. Preface: This does not really matter much.
But it would be cool to know whether this is supposed to work and it is not,
or if it’s NOT supposed to work so don’t worry about it…
Not worrying about it is fine, and it just means to be complete and not omit stuff as shorthand.

This was the same issue back in these posts:
WavLen<> Specified or not is weird
Prof knows maybe why

The issue is WavLen<> needing taking the EFFECT_XXX or not.
In the following test, it does NOT work without specifying the EFFECT. (in the style Editor this would show as EFFECT_NONE)

TransitionEffectL<TrConcat<
TrInstant,GreenYellow,TrFadeX<Percentage<WavLen<>,65>>>,EFFECT_CLASH>,

To work, it needs to be:

TransitionEffectL<TrConcat<
TrInstant,GreenYellow,TrFadeX<Percentage<WavLen<EFFECT_CLASH>,65>>>,EFFECT_CLASH>,

So the lesson learned I think without bothering with this too much is to just always provide the EFFECT and move on.
Unless it’s an actual under-the-hood thing that should be fixed?

So, obviously it would be nice if this worked.
I’m pretty certain the issue is that TrConcat gets in the way.
The link between TransitionEffectL and WavLen only exists for a single run() call, and TrConcat ends up calling run() on TrInstant when that happens, not the TrFadeX.
I think fixing it will have to be opportunistic, meaning that if I can figure out a better way, then I will fix it.

So yeah. Nice, but not necessary. Bigger fish to fry for sure. ok thanks.

but also just to note it’s not consistant.
The layer beneath for stab is working fine as just Wavlen<> ( here’s the not-simplified layer)

TransitionEffectL<TrConcat<TrInstant,GreenYellow,TrDelay<25>,AlphaL<Black,Int<0>>,TrWipeInX<Percentage<WavLen<>,30>>,AlphaL<Stripes<5000,1000,Orange,DarkOrange,Rgb<150,60,0>,Rgb<60,30,0>,Rgb<150,14,0>,OrangeRed>,SmoothStep<Int<20000>,Int<20000>>>,TrJoin<TrSmoothFadeX<Percentage<WavLen<>,90>>,TrWipeX<Percentage<WavLen<>,70>>>>,EFFECT_STAB>,
 

So I was taking a look at why this WavLen works but not the other one, and I realized that they really should both work. So now I’m investigating why the first one didn’t work instead…

Did some testing, and the layer seems to work for me…

Maybe something in context? because it’s Percentage?

This is where it was happening to me.

This is an effects layer pack I use, and it’s where the clash layer wasn’t working with just WavLen<>.
Mind you, the first part of the TrConcat DID show (the GreenYellow flash for 25ms), but none of the localized impact that follows worked.
Again, specifying the EFFECT_CLASH makes it work.

It’s most likely due to EFFECT_CLASH_UPDATE being last detected.

1 Like

Wonder if maybe there are quite a few EFFECT_CLASH_UPDATE events.
Blades only remember the last 7 events, so maybe the clash effect ends up getting forgotten.

No… that’s not it. EFFECT_CLASH_UPDATE isn’t stored in the blades effects_ array.

I tested the BC_effects_1, and it seems to work for me.
Unless I just magically fixed when I made WavLen an SVF somehow…

Ok. I’ve extensively tested this now, even with the new WavLen as SVF update, it’s not working 100%.

The issue seems to be something about using regular TransitionEffect<> as the effect COLOR inside the layer TransitionEffectL<>.

Here are my tests and results.

// this works (no nested TransitionEffect<>, no EFFECT specified)
TransitionEffectL<TrConcat<TrInstant,Red,TrFadeX<WavLen<>>>,EFFECT_CLASH>,

// this works on its own not nested in a TransitionEffectL wrapper.
TransitionEffect<Red,White,TrInstant,TrFadeX<WavLen<>>,EFFECT_CLASH>,

// This is not working: nesting the TransitionEffect<> as COLOR, still no EFFECT specified anywhere)
TransitionEffectL<TrConcat<TrInstant,TransitionEffect<Red,White,TrInstant,TrFadeX<WavLen<>>,EFFECT_CLASH>,
																																					 TrFadeX<WavLen<>>>,EFFECT_CLASH>,

// This works when adding specified EFFECT only in TransitionEffectL wrapper.
TransitionEffectL<TrConcat<TrInstant,TransitionEffect<Red,White,TrInstant,TrFadeX<WavLen<>>,EFFECT_CLASH>,
                                                          								 TrFadeX<WavLen<EFFECT_CLASH>>>,EFFECT_CLASH>,

// but not working with EFFECT specified in nested TransitionEffect<> only.
TransitionEffectL<TrConcat<TrInstant,TransitionEffect<Red,White,TrInstant,TrFadeX<WavLen<EFFECT_CLASH>>,EFFECT_CLASH>,
                           			                          							 TrFadeX<WavLen<>>>,EFFECT_CLASH>,

// This also works without 2nd WavLen in TransitionEffectL wrapper.
TransitionEffectL<TrConcat<TrInstant,TransitionEffect<Red,White,TrInstant,TrFadeX<WavLen<>>,EFFECT_CLASH>,
																																			  	 TrDelay<2000>>,EFFECT_CLASH>,

This makes sense I think.
TransitionEffect(L) doesn’t save the previous value of the found effect, so when it pops back out, the found effect becomes null. Should be relatively easy to fix.
I don’t think I ever imagined anybody putting a TransitionEffect inside a TransitionEffect…