-
-
Notifications
You must be signed in to change notification settings - Fork 264
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
Adding support to the Keysight N5242B VNA #1051
Comments
@cafeclimber , do you have any comment/idea ? |
Hey yeah! So if you look at the source for the PNA class, it contains a list of supported models. That is just a list of models that have been tested by maintainers to ensure all commands work as expected. If you're able to run through all the command's available to your PNA, and everything works you can submit a PR adding that model to the list! You mentioned you have a PNA-X, I'm not sure if that introduces additional functionality that would need to be added, but if so, we can discuss potentially introducing another class for PNA-X models. |
I believe the PNA-X adds a few additional features (found in this brochure and in points 2,5,6,8) but I am not a trained RF engineer, so I would be uncomfortable to set up the appropriate VISA test commands. If someone were to send the appropriate test sequence, I will be happy to run that. However, I can run through all the commands from the PNA class and verify that they are all functional. It's a little busy over here, so there will probably be about a week to get back to the system. |
I recently just learned about this package yesterday and have been really enjoying all of the features. I currently have access to a Keysight N5242B PNAX and got a warning when connecting using the package:
"This model (N5242B) has not been tested with scikit-rf. By default, all features are turned on but older instruments might be missing SCPI support for some commands which will cause errors."
I haven't had any issues yet, but would love to contribute to this project. Let me know if there are any test-scripts that I can run to verify your code. Thanks!
The text was updated successfully, but these errors were encountered: