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 support for NanoVNA F V2 for reaching max datapoints up to 201 and 301 on different firmwares #760

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

Conversation

xros
Copy link
Contributor

@xros xros commented Jan 19, 2025

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?

  • only work at max datapoints up to 101
  • incorrect SN code

Issue Number: #691

What is the new behavior?

  • max datapoints on NanoVNA F V2 can reach up to 301 starting from firmware 0.5.0 and onwards.

  • max datapoints on NanoVNA F V2 can reach up to 201 starting from firmware 0.2.0 and onwards.

  • Recommend NanoVNA F V2 owners to upgrade their firmware higher or equally than version 0.5.0.

Does this introduce a breaking change?

  • Yes
  • No

Other information

@zarath
Copy link
Collaborator

zarath commented Feb 1, 2025

Are you able to refactor your code contributions?

@xros
Copy link
Contributor Author

xros commented Feb 4, 2025

Are you able to refactor your code contributions?

It looks the root code base have fully refactored after my pull-request. When I made the pull-request, the code base was okay and working. I will have a look when I have time.

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.

2 participants