ho-ro/nanovna-tools

Toolbox for NanoVNA and tinySA: Small programs for scripting and automatisation

nanovna-tools's Issues

png.h is missing on the first make invocation

Where did you take png.h from? Any hints in the Readme welcome.
Thaks.

nanovna_capture.py: NameError: name 'bytestream' is not defined

@krupis reported in NanoVNA-Saver/nanovna-saver#676:

C:\Users\petrikas.lu\Downloads\nanovna-tools-main\nanovna-tools-main>python nanovna_capture.py
Traceback (most recent call last):
  File "C:\Users\petrikas.lu\Downloads\nanovna-tools-main\nanovna-tools-main\nanovna_capture.py", line 115, in <module>
    if bytestream == b'capture?\r\nch> ': # error message
       ^^^^^^^^^^
NameError: name 'bytestream' is not defined

FIX: change bytestream to captured_bytes in this line

if len( captured_bytes ) != 2 * size:
    if bytestream == b'capture?\r\nch> ': # error message
        print( 'capture error - does the device support the "capture" cmd?' )
    else:
        print( 'capture error - wrong screen size?' )
    sys.exit()

capture error - wrong screen size using nanovna_capture.py (Windows 10, NanoVNA H)

We have discussed about this a little bit:
NanoVNA-Saver/nanovna-saver#676

After I have replaced:

#if bytestream == b'capture?\r\nch> ': # error message

with:

if captured_bytes == b'capture?\r\nch> ': # error message

to fix my initial issue:

 NameError: name 'bytestream' is not defined

I still cannot get to run this script on NanoVNA -H due to the following error:

C:\Users\petrikas.lu\Downloads\nanovna-tools-main\nanovna-tools-main>python nanovna_capture.py
capture error - wrong screen size?

Have you tried to run this on NanoVNA-H? Is that an issue related to NanoVNA - H? I tried using --help but does not seem to give any information regarding screen size.