Skip to content

phMod types - #333

Open
j-enns wants to merge 6 commits into
sensorium:masterfrom
j-enns:master
Open

j-enns wants to merge 6 commits into
sensorium:masterfrom
j-enns:master

Conversation

@j-enns

@j-enns j-enns commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

add
int8_t phMod(SFix<NI,NF,RANGE> phmod_proportion) and
int8_t phMod(SFix<15,16> phmod_proportion) to metaoscil

include other phMod types
make last value unnecessary, as in the comments
otherwise if final freq is left off this is always true
update setFreq to use last oscil for all frequencies above the penultimate cutoff, regardless of if the last oscillator has a cutoff frequency set
@j-enns

j-enns commented Oct 17, 2026

Copy link
Copy Markdown
Contributor Author

only commit c7479c6 matters

@tomcombriat

Copy link
Copy Markdown
Collaborator

Looks good to me, correcting something that got forgotten when transitioning to FixMath.

Reading it, I ended up wondering if the SFix<15,16> is actually useful, and in Oscil also, but that can be trimmed later if needed.

Would be good if you can squash all these commits into one.

Thanks!

This branch has not been deployed

No deployments
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.

2 participants