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

Filter frequency of all three synths set to 100 #23

Merged
merged 1 commit into from
Jul 20, 2024

Conversation

bratpeki
Copy link
Contributor

@bratpeki bratpeki commented Jul 15, 2024

To add to #22, the filter frequency of ADD, SUB and PAD have all been set to 100.

In LMMS, the filter frequency has been set to 127, the maximum value. This, according to Lost, allows for automating the filter as well as disabling the lowpass which happens by default.

The linked PR in LMMS is here.

@tresf
Copy link
Member

tresf commented Jul 15, 2024

Adding @LostRobotMusic and @JohannesLorenz as reviewers.

@bratpeki
Copy link
Contributor Author

To completely disable the lowpass, the default value of the MIDI controller for the filter cutoff should be moved to a value of 127. This makes the filter knob in the Zyn LMMS UI work as well, both manually and through automation, since it just updates the MIDI controller.

Copy link
Contributor

@JohannesLorenz JohannesLorenz left a comment

Choose a reason for hiding this comment

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

Approved.

Please formulate a commit message (like in the other PR).

In this repo, the synths (ADD, SUB and PAD) have had the filter
frequency changed to 100. Controller defaults have also been
changed (filtercutoff.depth).
@bratpeki
Copy link
Contributor Author

bratpeki commented Jul 19, 2024

I'm sorry for misunderstanding your comment. The commit message here is:

Made it so ZynAddSubFx isn't lowpassed by default

In this repo, the synths (ADD, SUB and PAD) have had the filter
frequency changed to 100. Controller defaults have also been
changed (filtercutoff.depth).

@JohannesLorenz JohannesLorenz merged commit 9499523 into LMMS:master Jul 20, 2024
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.

4 participants