Skip to content

Add Mutable Reverb#1065

Merged
stellar-aria merged 10 commits into
SynthstromAudible:communityfrom
stellar-aria:mutable-reverb
Jan 26, 2024
Merged

Add Mutable Reverb#1065
stellar-aria merged 10 commits into
SynthstromAudible:communityfrom
stellar-aria:mutable-reverb

Conversation

@stellar-aria
Copy link
Copy Markdown
Collaborator

@stellar-aria stellar-aria commented Jan 25, 2024

Closes #952

@soymonitus
Copy link
Copy Markdown
Collaborator

The new reverb model is astonishing. Thanks a lot.
I find though that changing Mode also resets the Roomsize, dampening and width. Is it really needed to reset them? If i have setup a "large" reverb with high values for those params, I would not want them to be changed if i change just the reverb model. Or is there any good reason to do it?

Comment thread src/deluge/gui/ui/menus.cpp Outdated
@soymonitus
Copy link
Copy Markdown
Collaborator

Small 'nit', in the global menu, the order is AMOUNT, MODEL, ROOM SIZE, while in the clip menu the order is MODEL, AMOUNT, ROOM SIZE. Could you make them consistent with each other?

Comment thread src/deluge/gui/ui/menus.cpp Outdated
Copy link
Copy Markdown
Collaborator

@seangoodvibes seangoodvibes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx for making the changes requested :)!

Comment thread src/deluge/dsp/reverb/base.hpp Outdated
Comment thread src/deluge/dsp/reverb/mutable/reverb.hpp
Comment thread src/deluge/dsp/reverb/mutable/reverb.hpp Outdated
@stellar-aria stellar-aria added this pull request to the merge queue Jan 26, 2024
Merged via the queue into SynthstromAudible:community with commit 3099ff1 Jan 26, 2024
@stellar-aria stellar-aria deleted the mutable-reverb branch January 26, 2024 19:36
tastycode pushed a commit to tastycode/DelugeFirmware that referenced this pull request Jul 3, 2024
* Add Mutable Reverb

* Point to updated argon

* Persist Reverb settings across model changes

* Add reverb model menu to global reverb menu

* Change model position in original menu

* change to deluge::vector

* Remove unnecessary setters

* Change case of process method
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.

Port Clouds reverb model

4 participants