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 Live View for viewing monitor screen of NanoVNA and TinySA in real time #762

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

Conversation

xros
Copy link
Contributor

@xros xros commented Jan 20, 2025

add Live View for viewing monitor screen of NanoVNA and TinySA in real time

This is another pull-request onwards after my previous pull-request #761 , the key commit is here 4838f0e

Pull Request type

Please check the type of change your PR introduces:

  • [] Bugfix
  • Feature
  • [] Code style update (formatting, renaming)
  • [] Refactoring (no functional changes, no API changes)
  • [] Build-related changes
  • [] Documentation content changes
  • [] Other (please describe):

What is the current behavior?

Not having

Issue Number: N/A

What is the new behavior?

  • Able to view nanovna/tinySA screen in real time using nanovna-saver
  • Tested on NanoVNA-F V3 and TinySA Ultra+ 407

Does this introduce a breaking change?

  • [] Yes
  • No

Other information

This Live View is still under development which might have some issues. Currently it can fresh the monitor every 2000ms but the transfer speed is too slow, sometimes it got frozen and the buffer for the image should be improved. That's because every screenshot will interrupt the current scan of nanovna or tinysa. Because nanovna or tinysa won't scan very fast. But anyway we have this functionality. Better than nothing.

Appreicate your contributions if you can

Xnip2025-01-20_14-30-22

By @xros , enjoy!

@xros xros requested a review from zarath as a code owner January 20, 2025 06:54
…ng screen data via USB from nanovna and tinysa. performance improved a little.
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.

1 participant