Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added capability of customising minimum amplitude level of enclosures added internally by GrandOrgue https://github.com/GrandOrgue/grandorgue/issues/782 #2123

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

oleg68
Copy link
Contributor

@oleg68 oleg68 commented Feb 17, 2025

#Resolves: #782

This PR adds an Enclosures tab to the Organ Settings dialog.

It shows the Minimal amplitude level of all enclosures and allows the user to change them for all internal enclosures.

@larspalo
Copy link
Contributor

@oleg68 On Ubuntu 20.04 the build of this PR will crash immediately when I try to open the Audio/Midi->Organ Settings dialog. It doesn't matter what sample set I try to use or what settings I might have changed, GO reliably crashes as soon as I open the Organ Settings dialog.

Fixed help window appearence

Updated help

Fixed buttons on the Settings dialog

Fixed compilation errors

Fixed "Apply or discard first"

Fixed pre-commit

Intermediate commit

Intermediate commit
@oleg68 oleg68 force-pushed the feature/organ-settings-enclosures2 branch from b3b92ff to a4dad85 Compare February 22, 2025 12:12
@oleg68
Copy link
Contributor Author

oleg68 commented Feb 22, 2025

@larspalo I fixed the crash

@oleg68
Copy link
Contributor Author

oleg68 commented Feb 27, 2025

@larspalo @rousseldenis could you approve this PR?

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.

Add setting of Enclosure AmpMinimumLevel to the GUI
2 participants