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

Phase #7220

Open
baconpaul opened this issue Sep 23, 2023 · 2 comments
Open

Phase #7220

baconpaul opened this issue Sep 23, 2023 · 2 comments
Labels
DSP Issues and feature requests related to sound generation in the synth Feature Request New feature request Oscillator Features or additions to the oscillator section
Milestone

Comments

@baconpaul
Copy link
Collaborator

Oh ha ha ha ha.
surge-synthesizer/surge-rack#937 (comment)

So this gave me an idea. The reason we haven't added initial phase to surge is a couple of fold, but one of them is there's no place for a param. But there is a place in teh API. And we want this for XT2. So I could quite easily do this

  1. Change oscillator::init to have a float initialPhase=0 argument
  2. Visit all the oscillators and implement it. Implementing it is trivial.
  3. Don't expose that to the surge VST in the 1.x series. Sorry. Sucks to be you VST users
  4. But expose it in the rack modules since they call ::init directly and aren't bound by the param infrastructure in the same way then
  5. In XT2 add a initial-phase param which just blats onto that setting
@baconpaul baconpaul added the Feature Request New feature request label Sep 23, 2023
@baconpaul baconpaul added this to the Surge XT 1.3 milestone Sep 23, 2023
@mkruselj
Copy link
Collaborator

We actually have issue #867 for this, hehe 🙂

@baconpaul
Copy link
Collaborator Author

Yeah I was gonna find that

that’s for phase in surge and so should stay open even once this is closed but it will be way easier once I do this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DSP Issues and feature requests related to sound generation in the synth Feature Request New feature request Oscillator Features or additions to the oscillator section
Projects
None yet
Development

No branches or pull requests

2 participants