Author Topic: [available in v1.08] global fx attempt : REVERB  (Read 15209 times)

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #30 on: August 19, 2021, 11:49:33 PM »
it has some flaws, i fear ; the reverb saturate too easily (depending on the sound used).
i'm working on this issue, but with not much luck so far.

matrix12x

  • Team Member
  • ***
  • Posts: 188
    • View Profile
Re: global fx attempt
« Reply #31 on: August 20, 2021, 03:03:13 AM »
I was coming across the saturation, I just dial down the sends or the instrument levels.

Maybe insert a limiter before the reverb.

Feature request: If you ever get a chance to add a pitch shifter in a feedback loop to the reverb with variable attenuation and pitch (Shimmer), that would be cool.


Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #32 on: August 31, 2021, 08:13:44 AM »
i did improve the reverb headroom with different kind of hp&lp filters, same as made by Valley audio, i think it's ok now (attached file).
@matrix12x, sorry for the feature request, i think i will let it as it is, my attempt for shimmer effect was not very convicing.
But if you wish, don't hesitate to set a reverb preset while all parameters are there, i would add yours to the list.

Last step is to allow saving the reverb parameters in the mixer file format, but i have troubles making it work...

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #33 on: August 31, 2021, 09:52:48 PM »
Thanks for the reverb work !
I'll have a look on friday (day off :) ).


Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #34 on: August 31, 2021, 10:57:49 PM »
Thank you Xavier.

There is a problem with code structure : the FxBus class is not in the right place, there is some refactoring to be done...

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #35 on: September 02, 2021, 09:58:23 PM »
here is a version with complete reverb parameters, for preset design ( sorry for that, there were 3 missing parameters ) + reverb main level adjustment.

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #36 on: September 02, 2021, 10:14:51 PM »

Thanks :)
Is the latest version available from your git repo ?

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #37 on: September 02, 2021, 11:17:57 PM »
i did not sync my repo:  it's now done :)

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #38 on: September 03, 2021, 09:52:34 PM »
Thanks a lot !!
I played with the reverb today and that's really awesome to be able to have a reverb inside the preenfm3.
Congrats and thanks for the hard work  :)

I worked on your modification with the idea to integrate it in the firmware.
The branch is here, and you can have a look at the latest commit to see the difference with your repo.
https://github.com/Ixox/preenfm3/commits/FXGlobalFX-PVig

I hope you won't be disapointed but for the moment, i only kept the preset in the global param.
I added a volume and an output slection.
These 3 values as well as the send for each instrument are saved with the Mixer on the SD card.
« Last Edit: September 03, 2021, 10:11:28 PM by Xavier »

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #39 on: September 04, 2021, 10:28:40 AM »

I changed my mind a little.

I plan to have an option in MENU>CONFIG : "Reverb param".
If set to "All", all params will show up in the global settings.

And i will save all params with the mixer and not the preset.

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #40 on: September 04, 2021, 11:48:01 AM »
hello Xavier,
many thanks for the code integration :)
your additions are very useful !

But, yes, i agree that linking all reverb parameters to the mixer does make sense.

cheers !
 ;D

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #41 on: September 04, 2021, 06:20:46 PM »

I had some time today and there's now an option to show all params :
  MENU>Config>Reverb params : Show or Hide.
Default value is Hide.

So out of the box, you only see the reverb preset.
But all params are available for users who wants that, after modifying the config.

My commit is here :
https://github.com/Ixox/preenfm3/commits/FXGlobalFX-PVig

Let me know if you see anything strange.

Thanks  :)


Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #42 on: September 04, 2021, 08:00:38 PM »
The hide/show solution is a great idea but some troubles so far :
well, no luck, the reverb do not produce any sound on my pmf3, i could not spot the problem so far :P.

About the preset param, i think it may be confusing for users :
one may not understand that changing preset will change some, not all, parameters of page 2 & 3 (if in display all mode).

I would separate clearly the two modes : no preset choosing in user mode.

If you wish to go this way i can redefine the presets, with all parameters.

« Last Edit: September 04, 2021, 08:50:05 PM by Toltekradiation »

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #43 on: September 04, 2021, 09:48:43 PM »
Yes there's a sound problem with the current version !!!  :o
What did i change at the last minute ?? Will try to find out.


When changing the preset, i apply all default values before the preset specific param.
So all params are reset if they were all changed.
« Last Edit: September 04, 2021, 09:57:58 PM by Xavier »

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2257
    • View Profile
Re: global fx attempt
« Reply #44 on: September 04, 2021, 10:28:33 PM »

I pushed a fix.
I moved where the fxBus.init() is called, it works better  :)