Skip to content

Steps tendency#176

Open
DaanVanVugt wants to merge 4 commits into
iterorganization:developfrom
DaanVanVugt:steps-tendency
Open

Steps tendency#176
DaanVanVugt wants to merge 4 commits into
iterorganization:developfrom
DaanVanVugt:steps-tendency

Conversation

@DaanVanVugt

Copy link
Copy Markdown
Contributor

Includes a commit from #175

Comment thread waveform_editor/tendencies/steps.py
Comment thread waveform_editor/tendencies/steps.py Outdated
- Rename is_string -> is_categorical so booleans are treated like strings
  (held as a zero-order-hold step rather than coerced to float and interpolated)
- Reject mixing categorical (string/boolean) and numeric values within a single
  waveform, since the gaps between them cannot be interpolated
- Document the constant tendency value types (float/int/categorical) in the docs
A piecewise-constant tendency that holds each value from its breakpoint until the
next, supporting non-numeric (string/boolean) values. Registered as 'steps'.

- start is inferred from the first breakpoint; an explicit end/duration may be
  given so the final value is held for a real duration (defaults to the last time)
- shared time/value validation and start/end resolution are factored out of
  PiecewiseLinearTendency via overridable _coerce_value / _resolve_time_bounds
  hooks, so the steps tendency only overrides what genuinely differs
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