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

Add plot bandwidth measurement to UI #175

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

Conversation

andreiva
Copy link

I thought it would be handy to see signal bandwidth in kHz when adjusting plot deviation. See screenshot
I used signals and slots to update the UI, for that I had to move Tuner instantiation to MainWindow. I have very little experience with QT, so I'm not sure if I did it right.

inspectrum-plot-bw

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

This would be really useful to have, but looks like it's out dated. @andreiva could you rebase it to make it easier for @miek to merge?

@andreiva
Copy link
Author

Sure, I will take a look at it.

It occurred to me, that it would be a lot easier to display BW directly in spectrogramplot. It would probably be doable right in spectrogramplot.cpp. For example above bw rectangle? Any opinions?
It was somehow ugly how I had to pass bw value to spectrogramcontrols.cpp

@luzpaz
Copy link

luzpaz commented Nov 1, 2023

Is there any chance this will ever be 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

6 participants