Skip to content

Added LPF filter to MI reverb - #2788

Merged
stellar-aria merged 30 commits into
SynthstromAudible:communityfrom
markyounq:community
Oct 14, 2024
Merged

Added LPF filter to MI reverb#2788
stellar-aria merged 30 commits into
SynthstromAudible:communityfrom
markyounq:community

Conversation

@markyounq

Copy link
Copy Markdown
Contributor

WIP adding LPF filter to the Mutable Instruments reverb engine.

Comment thread src/deluge/dsp/reverb/reverb.hpp
@markyounq
markyounq marked this pull request as ready for review October 10, 2024 08:50
@markyounq markyounq changed the title [DRAFT] Added LPF filter to MI reverb Added LPF filter to MI reverb Oct 10, 2024
@m-m-adams

Copy link
Copy Markdown
Collaborator

Could you add saving/loading code before we merge?

It would also be nice to have this as a feature on both reverbs as a follow up. It would require refactoring so that the specific reverb process function writes to an intermediate buffer where we run the filters

@github-actions

github-actions Bot commented Oct 10, 2024

Copy link
Copy Markdown
Contributor

Test Results

103 tests  ±0   103 ✅ ±0   0s ⏱️ ±0s
 15 suites ±0     0 💤 ±0 
 15 files   ±0     0 ❌ ±0 

Results for commit d026266. ± Comparison against base commit ace2163.

♻️ This comment has been updated with latest results.

Comment thread src/deluge/dsp/reverb/mutable/reverb.hpp Outdated
@markyounq

Copy link
Copy Markdown
Contributor Author

Could you add saving/loading code before we merge?

I'm not sure what you mean. Saving/loading the reverb LPF cutoff?

It would also be nice to have this as a feature on both reverbs as a follow up. It would require refactoring so that the specific reverb process function writes to an intermediate buffer where we run the filters

Agreed but I don't think I'll cover it in this PR

Comment thread src/deluge/dsp/reverb/base.hpp
@m-m-adams

Copy link
Copy Markdown
Collaborator

I'm not sure what you mean. Saving/loading the reverb LPF cutoff?

Yeah just needs to be added to the reverb block in the song-> read/write from file functions here

writer.writeAttribute("roomSize", roomSize);

Comment thread src/deluge/dsp/reverb/base.hpp Outdated
Comment thread src/deluge/dsp/reverb/base.hpp Outdated
Comment thread src/deluge/dsp/reverb/base.hpp Outdated
Comment thread src/deluge/dsp/reverb/mutable/reverb.hpp Outdated
Comment thread src/deluge/dsp/reverb/mutable/reverb.hpp Outdated
markyounq and others added 2 commits October 14, 2024 13:07
Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
seangoodvibes pushed a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 11, 2026
* Added LPF filter to MI reverb

* added LPF.h file

* Fixed LPF so it applies to both channels

* added initialization of reverb default

* testing defaults

* more LPF default testing. init LPF in base reverb

* fixed typo in LPF init

* more default testing

* edited song params for default reverb

* fixing calcfiltercutoff to work for both LPF and HPF ranges

* added param to function calls

* added filter type to scope of init

* changed min reverb LPF cutoff to 0hz

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/base.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* Update src/deluge/dsp/reverb/mutable/reverb.hpp

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>

* added save/write to song

* added template to reverb base

* edit LPF init

* added LPF to global kit FX

* edited comments to match new Hz

* updating docs

* ran format

---------

Co-authored-by: Katherine Whitlock <kate@skylinesynths.nyc>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants