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

Remote TCP Input - FFT does not fix offset on axis X #1377

Closed
moffa90 opened this issue Aug 5, 2022 · 5 comments
Closed

Remote TCP Input - FFT does not fix offset on axis X #1377

moffa90 opened this issue Aug 5, 2022 · 5 comments
Assignees
Labels

Comments

@moffa90
Copy link

moffa90 commented Aug 5, 2022

Hey nice work on RemoteTCP Sink. I was using it and i get notice that the FFT graph on axis X doesn't fix accordding to the offset i setup on the plugin parameters, always is centered on the device center frequency e.g:

  • Center frequency device: 482.500 kHz
  • Remote TCP sink offset: + 150.000 Hz

With these configuration the FFT graph is centering on 482.500, i guess it should be on 482.650 kHz right? or is this the behavior intended?. I am attaching an image of the escenario to support my comment in case i couldn't explain properly 🐱

Thank you... 👍

Screenshot_42

@srcejon
Copy link
Collaborator

srcejon commented Aug 5, 2022

In my mind at least, the frequency offset should only be used if decimation is enabled (and you have that set to 1 in the pic)

@moffa90
Copy link
Author

moffa90 commented Aug 5, 2022

Maybe i missed some details on my last post than can help. The original signal is 2Ms/s, i am down converting it to 48khz, the signal you see on the image is being generated on 482.650 kHz but the FFT graph is centering it on 482.500 where the SDR center freq is configured, so that is the reason i guess the offset should be considered. If we see the FFT it says to me that the signal is at 482.500kHz when is not.

@srcejon srcejon self-assigned this Aug 6, 2022
@srcejon srcejon added the bug label Aug 8, 2022
@srcejon
Copy link
Collaborator

srcejon commented Aug 8, 2022

Yep, didn't look closely enough at your screenshot. Will fix.

@moffa90
Copy link
Author

moffa90 commented Aug 8, 2022

Thank you... 👍

@f4exb
Copy link
Owner

f4exb commented Aug 14, 2022

Implemented in v7.6.2

@f4exb f4exb closed this as completed Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants