-
Notifications
You must be signed in to change notification settings - Fork 301
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: adding new driver for Agilent signal generator E8267C #597
Conversation
max_value=20e-9)) | ||
self.add_parameter(name='phase', | ||
label='Phase', | ||
unit='rad', |
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.
looks like this is degrees?
|
||
# functions to convert between rad and deg | ||
|
||
def deg_to_rad(self, angle_deg): |
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.
These functions are in numpy https://docs.scipy.org/doc/numpy/reference/generated/numpy.deg2rad.html perhaps use them from there
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.
definitely use numpy !
@jenshnielsen @giulioungaretti fixed the numpy functions |
Author: euchas <adomas.valantinas@gmail.com> driver: add Agilent signal generator E8267C (#597)
Author: euchas <adomas.valantinas@gmail.com> driver: add Agilent signal generator E8267C (#597)
added all basic commands.
why no one likes 馃拑 :(
@giulioungaretti @jenshnielsen