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

NanoVNA H-4, 401 sweep points #367

Closed
galileo-pkm opened this issue Jan 8, 2021 · 6 comments
Closed

NanoVNA H-4, 401 sweep points #367

galileo-pkm opened this issue Jan 8, 2021 · 6 comments
Labels
stale Long time no reaction, will be closed soon

Comments

@galileo-pkm
Copy link
Contributor

The Nano H4, due to larger memory, has support for 401 sweep points.
Saver does not seem to detect that. Can this be enabled?
This is with Dislords 1.45 firmware.

@galileo-pkm
Copy link
Contributor Author

It appears to be an easy fix, (valid_datapoints = (101, 11, 51, 201, 401) in NanoVNA_H4.py)
Pull request: #369

@jkrnl
Copy link

jkrnl commented Feb 21, 2021

Solution seems not to be stable. Occasionally throws error what dissapears when I switch back to 101
Please, could you check it?

2021-02-21 15:17:40,290 - NanoVNASaver.SweepWorker - ERROR - list index out of range Traceback (most recent call last): File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\SweepWorker.py", line 80, in run self._run() File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\SweepWorker.py", line 121, in _run freq, values11, values21 = self.readAveragedSegment( File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\SweepWorker.py", line 236, in readAveragedSegment freq, tmp11, tmp21 = self.readSegment(start, stop) File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\SweepWorker.py", line 264, in readSegment values11 = self.readData("data 0") File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\SweepWorker.py", line 279, in readData tmpdata = self.app.vna.readValues(data) File "c:\users\x\appdata\local\programs\python\python39\lib\site-packages\NanoVNASaver\Hardware\NanoVNA.py", line 147, in readValues f"{data[2]} {data[3]}")) IndexError: list index out of range

@galileo-pkm
Copy link
Contributor Author

Your firmware version?

@jkrnl
Copy link

jkrnl commented Feb 21, 2021

Your firmware version?
sorry:
DiSlord 1.0.45
MaybeI found a discrepancy on bandwidth selection, this fw offers :30/100/333/1000/2000 and 4000 Hz on device, but not all of these are shown on NanoVNA saver dropdown.
Thank you

@galileo-pkm
Copy link
Contributor Author

I have been using the same firmware and haven't seen any issues
but I rarely use anything other than 333Hz BW.
Anyway this change adds only a single array element, no actual code change
so the issue is somewhere else.

@github-actions
Copy link

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍
Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions bot added the stale Long time no reaction, will be closed soon label Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Long time no reaction, will be closed soon
Projects
None yet
Development

No branches or pull requests

2 participants