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

Adds Frequency Offset option to adjust the vertical freq to captured frequency center #169

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jwcrawley
Copy link

This is a feature to implement a frequency offset, found just below the sample rate. If at 0, Inspectrum will function as it has before, with the center at 0 Hz, with 1/2 sample rate being the positive and negative bounds (eg: -10MHz to 10MHz centered around 0 Hz)

This feature shifts the 0 to the frequency that a user captured. For example, if a user was capturing a 30MHz window centering around 915MHz, they would see:

930MHz (upper bound)
915MHz (center)
900MHz (lower bound)

We also adjust this in accordance to the frequency and the sample rate in accordance how Inspectrum normally runs. We cannot detect this automatically, since IQ captures do not record it.

@miek
Copy link
Owner

miek commented Oct 1, 2020

Sorry for the delay getting back to you. Thank you for this.
It all looks good to me, except for one nitpick: I think it'd be better to call it "Centre frequency". That matches what GNURadio and other tools call it, and should make it's purpose more obvious to users. What do you think?

@jwcrawley
Copy link
Author

jwcrawley commented Oct 2, 2020 via email

Base automatically changed from master to main March 1, 2021 15:37
@catkira
Copy link
Contributor

catkira commented Oct 19, 2022

@jwcrawley I think the feature is nice. Can You rebase Your PR to the latest master branch (now main ^^) so that it can be easily merged?

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.

None yet

3 participants