-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Raw s21 phase at low frequencies #81
Comments
Yes, measure S21 L/R pins swapped on AIC codec input, not possible change this by software |
Rechecking the schematic, the polarity of the signal path for mixers U6 and U8 appear correct, yet measurement of raw s21 saved by the firmware is of opposite phase to that expected as in the screenshot. Nevertheless, raw s21 is opposite phase to expected as show in the screenshot. Yes, correction masks the problem... but if one wants to extract raw measurements to correct elsewhere, then all the s21 figures have to been inverted to make sense. So the question is why is s21 inverted, and it seems as I understand it, the fix involves multiplying the raw Port 2 measurement complex value by -1, so real and imaginary parts each *-1... or undoing where they are incorrectly inverted in the current firmware? The discussion at An experiment with NanoVNA and series through impedance measurement shows that the 'corrected s21' from the firmware is flawed which leads to the reported values of series through R and X being wrong. Owen |
I double-checked everything and didn't find any errors. At the same time, when changing channel 1, there are no problems with the phase, but in channel 3 it is rotated 180 degrees, as if the differential inputs R and L were reversed. There is no way to programmatically connect them differently |
Thanks for the checking. Let me try to explain the schematic given earlier. Let's look at the AC voltage applied to mixer U6 (call it Vs), pin A is high, pin B is low so Va-Vb is approximately in phase with the CLK1 voltage. Let's put a short on the TX jack, so the AC voltage on mixer U7 (call it Vr) pin B is low and pin A is high... so Va-Vb is approximately in phase with the CLK1 voltage. Now this Vr is in phase with Vs, so to correctly calculate s11 (which is -ve for a short), the code must calculate -Vr/Vs. (The raw s11 saved to a .s1p file for short is correctly a -ve value.) Somewhere s11 calculation must include an inversion, as you note it is not in the schematic. Now look at mixer U8 used for s21. Va-Vb is approximately in phase with RX voltage, and in phase with TX voltage and CLK1 if Port 1 is connected to Port 2 with a very short cable... so no inversion is required to calculate s21. Perhaps what is happening is that both s11 and s21 calcs invert when only the s11 should invert (ie multiply by -1) and s21 should be non-inverting? |
It appears that this is fixed in v1.2.32??? Thanks. Owen |
where do I download version 1.32? I only see 1.2.29, thanks |
The latest released version from April 8th is still at 1.2.29. To get the very latest version you should build it on your own. Just follow the instructions of the #define VERSION "1.2.36" If you do not need the bleeding edge you can download the FW from my repo where I provide quite recent builds from DiSlord's original FW, you can get e.g. and select Download on the next page Currently at 1.2.33, will be updated during the next few days. And if you have an older -H without SD card you can use my modified FW that supports 8 calibration slots instead of the 5 original ones - current build is from DiSlord's latest version 1.2.36. |
thank you |
I am using NanoVVNA-H4 v4.3 and firmware v1.2.30.
I expected that raw (ie uncorrected) s21 phase would be close to 0°.
Instead, it is close to 180°.
Looking at the schematic:
The polarity of Port 2 connection to the mixer is opposite to Port 1 which I assume needs to be undone in the firmware... but is it?
Owen
The text was updated successfully, but these errors were encountered: