-
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add channeled version of TPS #642
Add channeled version of TPS #642
Conversation
Should probably have a notebook. We may want to just delete the old version? |
@jenshnielsen Yes, I think that noone ever used this driver, so we may just replace the un-channelised version with this one. |
self._instrument = instrument | ||
|
||
def calc_set_points(self): | ||
message = self._instrument.ask('WFMPre?') |
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.
There are some conditions under which this call does not return the full expected response. We should ensure that these conditions are not met. (Sorry, I don't remember the details, but it's something like an enabling of the channels)
|
||
class TPS2012WithChannels(VisaInstrument): | ||
""" | ||
This is the QCoDeS driver for the Tektronix 2012B oscilloscope. |
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.
The class name and docstring should agree; either this is THE QCoDeS driver for TPS2012 or it's the channelised version.
print(message) | ||
except VisaIOError: | ||
gotexception = True | ||
self.visa_handle.timeout = original_timeout |
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.
Missing newline.
22f1904
to
79ebcfb
Compare
Removed the old one and fixed som pep8 issues, We still need to fix the preamble issue |
if not indented the linter thinks DATa and Note are parameters
Otherwise name will be inconsistent with/without channel access
Author: Jens Hedegaard Nielsen <jenshnielsen@gmail.com> Add channeled version of TPS (#642)
Author: Jens Hedegaard Nielsen <jenshnielsen@gmail.com> Add channeled version of TPS (#642)
* Add channeled version of TPS * Remove old non channel version of TPS2012 driver * Docs: deconfuse linter if not indented the linter thinks DATa and Note are parameters * use propagated name in scopearray Otherwise name will be inconsistent with/without channel access
* Add channeled version of TPS * Remove old non channel version of TPS2012 driver * Docs: deconfuse linter if not indented the linter thinks DATa and Note are parameters * use propagated name in scopearray Otherwise name will be inconsistent with/without channel access
@WilliamHPNielsen