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

adding fft skip (zoom out) #187

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jacobagilbert
Copy link

This is an updated version of #96 I have been using for a bit. There was some discussion back then of the 'right' way to handle zooming out, and some concern that simply striding over the input data was not ideal but I'm submitting this again in case its useful because:

  1. You just can't do this right now at all
  2. Its super fast still because its not computing an FFT on every data point; one per visible column
  3. There is not currently a configuration panel or settings menu to select different decimation behavior
  4. I fixed the stuff that did not work correctly in the other PR (time-scale, sigmf annotations)

@devnulling
Copy link
Contributor

just tested this out, seems to work fairly well!

@Teque5
Copy link

Teque5 commented Sep 22, 2021

let's get it merged!

Signed-off-by: Jacob Gilbert <[email protected]>
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.

3 participants