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/AWG70000A broadbean compatibility #911
driver/AWG70000A broadbean compatibility #911
Conversation
makeSEQXFile should not take .wfmx files as its input, since that creates a dependence on having this module to be able to generate a valid input for makeSEQXFile
flat_wfmxs += [AWG70000A.makeWFMXFile(wfm, amplitude) | ||
for wfm in wfm_lst] | ||
|
||
# flat_wfmxs = [wfmx for lst in wfms for wfmx in lst] |
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.
Should this stay?
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.
Nope!
Codecov Report
@@ Coverage Diff @@
## master #911 +/- ##
=======================================
Coverage 78.63% 78.63%
=======================================
Files 33 33
Lines 4536 4536
=======================================
Hits 3567 3567
Misses 969 969 |
…amHPNielsen/Qcodes into driver/AWG70000A_broadbean_changes
Apart from the commented out line that should be removed this looks fine |
…amHPNielsen/Qcodes into driver/AWG70000A_broadbean_changes
…amHPNielsen/Qcodes into driver/AWG70000A_broadbean_changes
Author: William H.P. Nielsen <whpn@mailbox.org> driver/AWG70000A broadbean compatibility (#911)
To make the AWG70000A .seqx file generation play nicely with broadbean, some code restructuring and slight API changes are needed in the AWG70000A driver. This PR introduces those.
Changes proposed in this pull request:
makeSEQXFile
take numpy arrays rather than.wfmx
files as inputThere is a parallel PR in broadbean about this: QCoDeS/broadbean#47.
@jenshnielsen