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

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #45 on: September 05, 2021, 06:11:05 PM »
I did a pull request with some fix for the parameter value change.

while i'm at it, i found a bug on config save : it freeze the synth on save, i could not get much information on what happening : in debug mode, the error stack is :
Thread #1 [main] 1 [core: 0] (Suspended : Signal : SIGTRAP:Trace/breakpoint trap)   
   HardFault_Handler() at stm32h7xx_it.c:106 0x80351f6   
   <signal handler called>() at 0xffffffe9   
   dir_find() at ff.c:2 482 0x806719a   
   0x0   


Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: global fx attempt
« Reply #46 on: September 05, 2021, 09:16:25 PM »

Thanks for the PR.

The error is when you save the config ?
It works here.
Do you have a pfm3/settings.txt file on your SD ?
Can you save preset and mixers without problem ?
the dir_find in the stack is weird.



Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #47 on: September 05, 2021, 10:13:10 PM »
yes i can save mixer, default mixer, presets without problem.
I did not have pfm3/settings.txt on the SD,
so i made an empty one to test : freeze on config save
after reading the SD card again, the settings.txt is gone ???
so i got a working setting.txt from a pfm2, put it to the SD, reboot, save the config again : freeze again, and the settings.txt disappeared..

yes, quite weird.

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: global fx attempt
« Reply #48 on: September 05, 2021, 10:27:13 PM »

The settings file is removed than save again.
https://github.com/Ixox/preenfm3/blob/master/firmware/Src/filesystem/ConfigurationFile.cpp#L90

So i assume it fails when saving after having removed the file.
But i don't know why.


Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: global fx attempt
« Reply #49 on: September 05, 2021, 10:28:33 PM »
You can try with windows to copy your all pfm3 directory on your SD card.
Then rename the old one "pfm3.old", and the new one "pfm3".
Just to see if that's not the SD where it tries to save the file.

Or backup your SD card, format it and copy back your files.

Would be interesting to see if that makes any change.

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #50 on: September 05, 2021, 11:26:20 PM »
thanks for the ideas.
i tried but it did not solve the issue, even with deep formatting with the official SD formater.
I will try again tomorrow with a minimal SD card content.

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #51 on: September 06, 2021, 09:11:13 PM »
hmm, this problem is really weird :
i took another SD card, deep formated it, then put the official SD content (pfm3_SD.zip) on it.
With this SD, i could save every kind of files without any problem, ... but not the config  ???
if i follow the action in debug mode, it crash on line 89 in ConfigurationFile.cpp

well, i can still edit the SETTINGS.TXT by hand, so it's not such a big deal ::)

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: global fx attempt
« Reply #52 on: September 06, 2021, 10:21:33 PM »
2 things i have in mind :
1. Make sure you have correctly merged Menu.h and Menu.cpp files.
2. Clean the project & Rebuild ALL so that you don't have old enum in some object file.

I'll make a firmware binary tomorrow to make sure we use the same.

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: global fx attempt
« Reply #53 on: September 06, 2021, 11:45:04 PM »
Thanks for the support :D
you found the problem : i have some trouble with the cleaning of my project :
with the latest v0.107 build from github, everything work perfectly.
but with my local sources, even if i clean before building (Project>Clean in STM32CubeIde), i get the issue.

I work currently on a copy of your branch FXGlobalFX-PVig, which i only slightly modified.
Maybe I have a small drift in my project configuration, i will try to find out what's wrong.

Thanks !

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: global fx attempt
« Reply #54 on: October 15, 2021, 10:38:55 PM »
I built a release with the reverb : v1.08
See more details here :
https://github.com/Ixox/preenfm3/releases
Feedbacks are welcome,

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: [available in v1.08] global fx attempt : REVERB
« Reply #55 on: October 16, 2021, 05:32:06 PM »
Hello Xavier,
i think there is some bugs.

In the preset recall :
In my version, some values were smoothed on preset change ; that part of the code was called continuously to make it happen.
In your integration, you do call FxBus::paramChanged() once on preset change, so the smoothing code should be removed.
here is my commit about it :
https://github.com/pvig/preenfm3/commit/fa6be48fb8f49bd753005f473e1220ad24f88ff9

About the predelay time, lfo speed values,
i did some stupid code mistakes :-X, so these value change does not work anymore.
here is the commit :
https://github.com/pvig/preenfm3/commit/1ebce476547bbc72352021a21a66d388d906fc7a

While i was at it, i changed the order of some parameters in the reverb pages, i think it does more sense :
https://github.com/pvig/preenfm3/commit/bd56ea39c794f8f8ec84d0795f42096f0f69b2a6


Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: [available in v1.08] global fx attempt : REVERB
« Reply #56 on: October 17, 2021, 10:03:23 AM »
No problem  ;)
I'll make an other one with your bug fix.
Thanks a lot for the feedback !

Do you think you can get the latest master branch and make a pull request with your bug fix ?
If not i'll integrate myself, no problem.
« Last Edit: October 17, 2021, 10:47:40 AM by Xavier »

Toltekradiation

  • Sr. Member
  • ****
  • Posts: 254
    • View Profile
Re: [available in v1.08] global fx attempt : REVERB
« Reply #57 on: October 17, 2021, 10:34:29 AM »
ok, pull request done :)

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: [available in v1.08] global fx attempt : REVERB
« Reply #58 on: October 17, 2021, 10:47:23 AM »
Haha... That was quick.
Thanks a lot  :)

Xavier

  • Administrator
  • Hero Member
  • *
  • Posts: 2256
    • View Profile
Re: [available in v1.08] global fx attempt : REVERB
« Reply #59 on: October 17, 2021, 11:44:15 AM »
1.08a is available with those fixes.
Don't worry if you find others , this reverb is a big feature ;)