Skip to content

feat: add optional transition support to filter payloads - #50

Merged
ParrotXray merged 1 commit into
ParrotXray:mainfrom
Akinori107:main
Aug 26, 2026
Merged

ParrotXray merged 1 commit into
ParrotXray:mainfrom
Akinori107:main

Conversation

@Akinori107

Copy link
Copy Markdown
Collaborator
  • add optional transition param (dict, e.g. durationMs=1500) to Equalizer, Timescale, Karaoke, Tremolo, Vibrato, Rotation, ChannelMix, Distortion, LowPass, Echo, Chorus, Compressor, Highpass, Phaser and Spatial
  • add Filter._build() to construct self.payload and cut duplicated code across filter classes
  • fix LowPass applying transition only to the lowpass key and not lowPass, both now receive the same settings dict
  • transition lives on the Filter base, so update()/reset() handle it like any other param
  • eq now compares transition too, so a transition-only edit is correctly detected as a change
  • default behavior unchanged when transition is omitted
  • document the transition parameter in docs/hdi/filters.md

- add optional transition param (dict, e.g. durationMs=1500) to Equalizer, Timescale, Karaoke, Tremolo, Vibrato, Rotation, ChannelMix, Distortion, LowPass, Echo, Chorus, Compressor, Highpass, Phaser and Spatial
- add Filter._build() to construct self.payload and cut duplicated code across filter classes
- fix LowPass applying transition only to the lowpass key and not lowPass, both now receive the same settings dict
- transition lives on the Filter base, so update()/reset() handle it like any other param
- __eq__ now compares transition too, so a transition-only edit is correctly detected as a change
- default behavior unchanged when transition is omitted
- document the transition parameter in docs/hdi/filters.md
@Akinori107
Akinori107 requested a review from ParrotXray August 26, 2026 10:29

@ParrotXray ParrotXray left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, i have reviwed

@ParrotXray
ParrotXray merged commit 10a9637 into ParrotXray:main Aug 26, 2026
9 checks passed
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