Skip to content
This repository has been archived by the owner on May 18, 2021. It is now read-only.

feat(widgets): Implement Waveform - and Spectrogram widgets #11

Merged
merged 21 commits into from
Mar 2, 2021

Conversation

jantischhoefer
Copy link
Member

@jantischhoefer jantischhoefer commented Feb 5, 2021

Initial pull request for discussion and feedback on widget implementation.

TODOs:

  • handle resizes of application and resize
  • switch setInterval to reception of real amplitude from backend
  • maybe implement Buffer for more consistent data flow (paused)
  • cosmetic enhancement of widget (Axis ticks, grid, etc.)
  • support multiple waveforms
  • documentation
  • tests

@jantischhoefer jantischhoefer added the enhancement New feature or request label Feb 5, 2021
@jantischhoefer jantischhoefer self-assigned this Feb 5, 2021
@fussel178 fussel178 changed the title Initial waveform widget implementation feat(widgets): Initial waveform widget implementation Feb 25, 2021
@jantischhoefer jantischhoefer marked this pull request as ready for review February 27, 2021 15:00
@jantischhoefer
Copy link
Member Author

Basic functionality works and can be merged.

@pklaschka pklaschka changed the title feat(widgets): Initial waveform widget implementation feat(widgets): Implement Waveform - and Spectrogram widgets Mar 2, 2021
@pklaschka pklaschka merged commit 9178004 into main Mar 2, 2021
@pklaschka pklaschka deleted the feat-waveform branch March 2, 2021 02:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants