Surge: Wavetable Label on UI

Created on 5 Mar 2019  ·  10Comments  ·  Source: surge-synthesizer/surge

Good afternoon/day or evening folks.. i was just playing with the wavetable feature of surge and it struck me that ive found yet another example of a nomenclature within surge that isnt fit for purpose

Currently the Wavetable Section contains a parameter labeled "Shape" in order to describe effectively the traversal of wavetables from one to another.. it also displays this value as a % .. presumably a % of the total of 64 waveforms/waves in a wt.

wtdisplay

presuming there is a format for WT which there certainly is and presuming that format consists of a maximum of 64 0-63 wavetable files or oscillators ..

that parameter should ..

  1. display the current wavetable number ..
  2. be labeled something like "travel" or "morph" because thats actually what its doing .. its traversing the "wave: table"

thoughts?

bang up job boys..

Good First Issue

All 10 comments

“Shape” is the wrong name
I think the control interpolates between tables so it will be fractional. That is with 5 tables the 12.5% setting is .5 * table0 + .5 * table1 - although I agree that would ideally show table number of “1.5” perhaps? And also i need to check that my statement is true :)

@baconpaul i would advocate a leaning to a more traditional nomenclature .. such as "morph" ?

Good suggestion!

Renaming this is in src/dsp/WavetableOscillator.cpp at line 103 or so (in ::init_ctrltypes). Changing the name shouldn't break streamed patches (the stream as a_osc1_param1 - the name is just for display) but this change would confirm that also in a couple of hosts.

Oh and we should change this in both Wavetable and Window when we do it.

yes they both refer to "shape"

I have a branch that updates the label to morph. Do we want this to display something other than 0 - 100% morphed?

Im not sure what the correct answer would be.

"wt position" for label and then the wt # seems to be the correct thing TBH. But im not sure that is actually what is happening. Is it mixing between wt's or just switching?

Doing the formula that Paul suggested above would take a bit of work and Im not sure it's very clear.

What does a morph of 1.5 mean exactly? A wt being 0% morph'ed or 100% morph'ed seems fairly understandable.

Regardless, I can open a PR if all we want to do is update the label.

It mixes in wavetables mode and switches in window mode I bekeivr; the percentage is the percent along the table count for that wt

Changing to not a percent is hard. With lots of ramifications. Even though it may be correct eventually

Morph vs position vs count I don’t have an opinion

Given that updating the label to morph makes the most @sense-amr to me <-- See what I did there?!?!

I opened a PR for this #760

OK I'll merge it when I look at PRs and stuff tomorrow. Thanks @itsmedavep!!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

baconpaul picture baconpaul  ·  5Comments

sense-amr picture sense-amr  ·  10Comments

baconpaul picture baconpaul  ·  7Comments

jarkk0sakkinen picture jarkk0sakkinen  ·  5Comments

baconpaul picture baconpaul  ·  9Comments