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 snr map #1341

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Remote snr map #1341

wants to merge 16 commits into from

Conversation

grappas
Copy link

@grappas grappas commented Mar 12, 2024

According to feature request: #1173

Feature is not done yet. Do not merge it yet.
I need to figure out how to populate snr_map. Here's the question - in which function have you implemented SNR diagram? Tried to figure it out on my own, but no success to this moment.

@argilo
Copy link
Member

argilo commented Mar 13, 2024

Signal power data can be retrieved from the rx_fft_c block using the get_fft_data method:

int get_fft_data(float* fftPoints);

The receiver object provides a wrapper around this:

int get_iq_fft_data(float* fftPoints);

Currently, this data is retrieved by MainWindow::iqFftTimeout() to update the display periodically.

@grappas
Copy link
Author

grappas commented Mar 13, 2024

Thx. I'll see into it.

@argilo argilo added the feature label Mar 19, 2024
@grappas
Copy link
Author

grappas commented Apr 20, 2024

@argilo can you check it?

@grappas
Copy link
Author

grappas commented Apr 21, 2024

?

@grappas
Copy link
Author

grappas commented May 20, 2024

anytime now...

@willcode
Copy link
Contributor

Agree that the data should only be copied when requested.

@grappas
Copy link
Author

grappas commented May 21, 2024

It's not copied anymore. It's referenced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants