Skip to content

fix(effects): Sync native bridge effect implementations with updated …#712

Merged
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/fix-effects-count-display-2zDe5
Jan 6, 2026
Merged

fix(effects): Sync native bridge effect implementations with updated …#712
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/fix-effects-count-display-2zDe5

Conversation

@braedonsaunders
Copy link
Owner

…types

  • Update granular_delay.rs to use position field (renamed from delay_time)
  • Update harmonizer.rs to use HarmonizerScale enum instead of PitchCorrectionScale
  • Add all missing HarmonyType variants (Fourth, MinorThird, Sixth, etc.)
  • Add Allpass case to multi_filter.rs filter type match
  • Add Brake case to rotary_speaker.rs speed match
  • Add Square and Sawtooth waveforms to vibrato.rs

…types

- Update granular_delay.rs to use `position` field (renamed from delay_time)
- Update harmonizer.rs to use HarmonizerScale enum instead of PitchCorrectionScale
- Add all missing HarmonyType variants (Fourth, MinorThird, Sixth, etc.)
- Add Allpass case to multi_filter.rs filter type match
- Add Brake case to rotary_speaker.rs speed match
- Add Square and Sawtooth waveforms to vibrato.rs
@vercel
Copy link

vercel bot commented Jan 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
openstudio Ready Ready Preview, Comment Jan 6, 2026 1:33am

@braedonsaunders braedonsaunders merged commit d2dcd33 into claude/build-openstudio-app-kgia5 Jan 6, 2026
2 checks passed
@braedonsaunders braedonsaunders deleted the claude/fix-effects-count-display-2zDe5 branch January 6, 2026 02:17
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