On Mon, Sep 21, 2020 at 02:12 AM, OneOfEleven wrote:
Hi Eirk
I'm just adding the changes you've made into my code (with some changes).
Cathy
Erik,
? I just tried Cathy's latest main NanoVNA-App branch, which adds your changes for communication with the tinySA, and it does not have the marker placement issue.? You might want to merge her marker placement code into the NanoVNA-App for the tinySA.
?
? I also asked Cathy if she would change the vertical units to dBm instead of dB when in the tinySA mode, and? if she could remove s11/s21 from the chart when in the tinySA mode.? Could you consider doing the same in your branch?
?Cathy got the marker placement issue correct but her tinySA mode does not handle combining multiple segments properly.? Your branch does handle that correctly so be careful not to merge the incorrect segments code.
?- Herb
