-
Notifications
You must be signed in to change notification settings - Fork 333
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
Driver for the HP 8116A 50 MHz Pulse/Function Generator #559
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CodingMarco that looks very promising!
One thing I did not see is the check_errors
function, to see if the instrument reoirts any errors (if such a status or command is available on the instrument)
Another item, that you could still add is the hp8116A.rst in docs/api/instruments/hp/
For that you could copy the hp3478A.rst and change it to match for the 8116, also than please add the hp8116A.rst to the index.rst
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonderful work. Congratulations and thanks for contributing to pymeasure. Some really minor remarks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good contribution. Thanks 👍
This PR adds a driver for the HP 8116A. I have tested it as good as possible with two units, one with and one without option 001 (sweep/burst capability).