Decadac doesn't get correct voltages on import #563
Comments
spauka
added a commit
to spauka/Qcodes
that referenced
this issue
Apr 11, 2017
Commits a channelized version of the Harvard DecaDAC driver using the new channelization implementation. Note that this driver at the present time also fixes a number of open bugs in the previous driver, including occasional crashes (microsoft#546), incorrectly reading DAC voltages (microsoft#563), reset (microsoft#478). This commit also adds a number of new features to the driver including: - Feature and version detection - Access to calibrated high-resolution mode of DAC - Parameter read from memory - Validation of writes Closes: microsoft#478, microsoft#543, microsoft#563
spauka
added a commit
to spauka/Qcodes
that referenced
this issue
Apr 11, 2017
Commits a channelized version of the Harvard DecaDAC driver using the new channelization implementation. Note that this driver at the present time also fixes a number of open bugs in the previous driver, including occasional crashes (microsoft#546), incorrectly reading DAC voltages (microsoft#563), reset (microsoft#478). This commit also adds a number of new features to the driver including: - Feature and version detection - Access to calibrated high-resolution mode of DAC - Parameter read from memory - Validation of writes
2 tasks
|
There is an implementation of a channelized version of the DecaDAC that solves this issue that you can see at spauka/Qcodes@76af2a3 that solves this issue. You may want to backport the code that reads the DAC memory depending on your needs. |
|
Thanks @spauka. Just had a look and it looks fab! It looks like you rewrote most of the driver ;) . I didn't see how that particular commit would fix my problem but that could well be down to my shaky DecaDAC knowledge. Hopefully can be merged soon and I'll try it out next time I'm using the DecaDAC :) |
jenshnielsen
added a commit
that referenced
this issue
Jun 16, 2017
* Driver/Keysight DMM (#556) * feat: Add a Keysight_34465A driver Add a first file for a driver for the Keysight 34465A DMM and a benchmarking notebook * Add trigger parameters to driver Add four trigger parameters to the driver. * refactor: Remove 'READ?' as volt get_cmd Make a more general get_cmd for the volt parameter. Using simply 'READ?' is wrong in many cases. * feat: Add sampling parameters Add four parameters for sampling. * fix: Fix wrong volt parameter get_cmd * WIP: Add Array parameter * feat: Add functioning data buffer parameter * Move notebook * Respond to review * feat: Add channelization Initial commit of two new classes to allow for channelization of instruments. This includes ChannelList, a container for channels in an instrument, and InstrumentChannel which implements a base class for an instance of a channel. * feat: Add channelized Lakeshore Model336 Driver Add a driver for the Lakeshore Model 336 temperature controller using new channelization implementation. * feat: Add a channelized driver for the Harvard DAC Commits a channelized version of the Harvard DecaDAC driver using the new channelization implementation. Note that this driver at the present time also fixes a number of open bugs in the previous driver, including occasional crashes (#546), incorrectly reading DAC voltages (#563), reset (#478). This commit also adds a number of new features to the driver including: - Feature and version detection - Access to calibrated high-resolution mode of DAC - Parameter read from memory - Validation of writes * fix: bug in slicing and adding code Fix bug in slicing/adding code, not correctly passing along channel list name. * style: fix formatting and make codacy fixes * style: fix more code style issues - Removed useless methods - Whitespace * fix: flush is now handled correctly in instrument base class * feat: Allow channellists to be excluded from snapshots Allows certain channel lists to be excluded from snapshots. This is helpful when, for example, there are multiple ways to access the same channel. * fix: Allow snapshots of ChannelLists This feature is implemented by creating the idea of a submodule in the instrument class. This is just a class which implements the Instrument class, and may be a channel list or may just implement a logical collection of parameters within a parent instrument. For example, looking at the Lakeshore Model_336 driver, each channel can either be accessed as a member of a channel list, or by using the name of the channel: ```python3 therm = Model_336("therm", "...") # These two accesses are equivalent therm.A.temperature() therm.channels[0].temperature() ``` * style: remove force arg in snapshot_base Should think about the best way to implement this change. * style: remove unessecary import * fix: error in type check. * fix: submodules should be Metadatable Submodule objects must be Metadatables at minimum, instrument is too restrictive for channels. * Start adding tests for channels * Extend tests * Fix: ensure consistent dataset names regardless of chan[] or chanA reading * More work on channels testing * Make sure that instument is closed in tests * Depend on hypothesis for test * fix improve testing of channels * test more: * fix: channels pep8 * fix: ensure that channels can be extended from generator * More tests of adding channels * Add missing functions and parameters to channellist * fix: function testS * assert to unittest style * Revert "fix: function testS" This reverts commit ff8a316. * Revert "Add missing functions and parameters to channellist" This reverts commit 347c0b2. * Remove docs of non existing attributes * Docs: decadac quote *IDN to remove warning * Add some channel notebooks * fix: test_channels small tweeks * Add support to array parameters in channels And error clearly if you try to measure multiple multiparameters * Mock parameters add instruments to fix names in tests * refactor tests to reduce code duplication * Revert "Add some channel notebooks" This reverts commit 2dbacc0. * feat: Add channelised QDac driver Add a QDac driver using the new channelisation feature * feat: Add a set method to MultiChannelInstrumentParameter * docs: Add a driver example for the channelised Qdac * feat: Make ChannelList take default paramclass Make the paramclass returned by __getattr__ be customisable. * feat: Add a channelised QDac driver Add a channelised driver for the QDac. This requires changing the base code for channels a bit to allow for custom MultiChannelInstrumentParameter subclasses. * docs: Add notebook for QDac w. channels * style: Remove unnecessary comment * Fix syntax/spelling errors in DAC driver * Fix: qdac_channels remove magic number * Doc: clarify doc string * Fix: rename paramclass to be more specific * Lakeshore tab to space * pep8 lakeshore driver * Pep8 decadac driver * Fix warnings in qdac driver * Annotate parameters that linter has problems with
|
Now that @spauka's Decadac driver got merged in, can we close this? @nataliejpg, are you happy? |
|
@WilliamHPNielsen yes and also excited to try channels ;P |
|
@nataliejpg Oh, they're fantastic. |
Dominik-Vogel
pushed a commit
to Dominik-Vogel/Qcodes
that referenced
this issue
Aug 9, 2017
* Driver/Keysight DMM (microsoft#556) * feat: Add a Keysight_34465A driver Add a first file for a driver for the Keysight 34465A DMM and a benchmarking notebook * Add trigger parameters to driver Add four trigger parameters to the driver. * refactor: Remove 'READ?' as volt get_cmd Make a more general get_cmd for the volt parameter. Using simply 'READ?' is wrong in many cases. * feat: Add sampling parameters Add four parameters for sampling. * fix: Fix wrong volt parameter get_cmd * WIP: Add Array parameter * feat: Add functioning data buffer parameter * Move notebook * Respond to review * feat: Add channelization Initial commit of two new classes to allow for channelization of instruments. This includes ChannelList, a container for channels in an instrument, and InstrumentChannel which implements a base class for an instance of a channel. * feat: Add channelized Lakeshore Model336 Driver Add a driver for the Lakeshore Model 336 temperature controller using new channelization implementation. * feat: Add a channelized driver for the Harvard DAC Commits a channelized version of the Harvard DecaDAC driver using the new channelization implementation. Note that this driver at the present time also fixes a number of open bugs in the previous driver, including occasional crashes (microsoft#546), incorrectly reading DAC voltages (microsoft#563), reset (microsoft#478). This commit also adds a number of new features to the driver including: - Feature and version detection - Access to calibrated high-resolution mode of DAC - Parameter read from memory - Validation of writes * fix: bug in slicing and adding code Fix bug in slicing/adding code, not correctly passing along channel list name. * style: fix formatting and make codacy fixes * style: fix more code style issues - Removed useless methods - Whitespace * fix: flush is now handled correctly in instrument base class * feat: Allow channellists to be excluded from snapshots Allows certain channel lists to be excluded from snapshots. This is helpful when, for example, there are multiple ways to access the same channel. * fix: Allow snapshots of ChannelLists This feature is implemented by creating the idea of a submodule in the instrument class. This is just a class which implements the Instrument class, and may be a channel list or may just implement a logical collection of parameters within a parent instrument. For example, looking at the Lakeshore Model_336 driver, each channel can either be accessed as a member of a channel list, or by using the name of the channel: ```python3 therm = Model_336("therm", "...") # These two accesses are equivalent therm.A.temperature() therm.channels[0].temperature() ``` * style: remove force arg in snapshot_base Should think about the best way to implement this change. * style: remove unessecary import * fix: error in type check. * fix: submodules should be Metadatable Submodule objects must be Metadatables at minimum, instrument is too restrictive for channels. * Start adding tests for channels * Extend tests * Fix: ensure consistent dataset names regardless of chan[] or chanA reading * More work on channels testing * Make sure that instument is closed in tests * Depend on hypothesis for test * fix improve testing of channels * test more: * fix: channels pep8 * fix: ensure that channels can be extended from generator * More tests of adding channels * Add missing functions and parameters to channellist * fix: function testS * assert to unittest style * Revert "fix: function testS" This reverts commit ff8a316. * Revert "Add missing functions and parameters to channellist" This reverts commit 347c0b2. * Remove docs of non existing attributes * Docs: decadac quote *IDN to remove warning * Add some channel notebooks * fix: test_channels small tweeks * Add support to array parameters in channels And error clearly if you try to measure multiple multiparameters * Mock parameters add instruments to fix names in tests * refactor tests to reduce code duplication * Revert "Add some channel notebooks" This reverts commit 2dbacc0. * feat: Add channelised QDac driver Add a QDac driver using the new channelisation feature * feat: Add a set method to MultiChannelInstrumentParameter * docs: Add a driver example for the channelised Qdac * feat: Make ChannelList take default paramclass Make the paramclass returned by __getattr__ be customisable. * feat: Add a channelised QDac driver Add a channelised driver for the QDac. This requires changing the base code for channels a bit to allow for custom MultiChannelInstrumentParameter subclasses. * docs: Add notebook for QDac w. channels * style: Remove unnecessary comment * Fix syntax/spelling errors in DAC driver * Fix: qdac_channels remove magic number * Doc: clarify doc string * Fix: rename paramclass to be more specific * Lakeshore tab to space * pep8 lakeshore driver * Pep8 decadac driver * Fix warnings in qdac driver * Annotate parameters that linter has problems with
peendebak
pushed a commit
to VandersypenQutech/Qcodes
that referenced
this issue
Aug 11, 2017
* Driver/Keysight DMM (microsoft#556) * feat: Add a Keysight_34465A driver Add a first file for a driver for the Keysight 34465A DMM and a benchmarking notebook * Add trigger parameters to driver Add four trigger parameters to the driver. * refactor: Remove 'READ?' as volt get_cmd Make a more general get_cmd for the volt parameter. Using simply 'READ?' is wrong in many cases. * feat: Add sampling parameters Add four parameters for sampling. * fix: Fix wrong volt parameter get_cmd * WIP: Add Array parameter * feat: Add functioning data buffer parameter * Move notebook * Respond to review * feat: Add channelization Initial commit of two new classes to allow for channelization of instruments. This includes ChannelList, a container for channels in an instrument, and InstrumentChannel which implements a base class for an instance of a channel. * feat: Add channelized Lakeshore Model336 Driver Add a driver for the Lakeshore Model 336 temperature controller using new channelization implementation. * feat: Add a channelized driver for the Harvard DAC Commits a channelized version of the Harvard DecaDAC driver using the new channelization implementation. Note that this driver at the present time also fixes a number of open bugs in the previous driver, including occasional crashes (microsoft#546), incorrectly reading DAC voltages (microsoft#563), reset (microsoft#478). This commit also adds a number of new features to the driver including: - Feature and version detection - Access to calibrated high-resolution mode of DAC - Parameter read from memory - Validation of writes * fix: bug in slicing and adding code Fix bug in slicing/adding code, not correctly passing along channel list name. * style: fix formatting and make codacy fixes * style: fix more code style issues - Removed useless methods - Whitespace * fix: flush is now handled correctly in instrument base class * feat: Allow channellists to be excluded from snapshots Allows certain channel lists to be excluded from snapshots. This is helpful when, for example, there are multiple ways to access the same channel. * fix: Allow snapshots of ChannelLists This feature is implemented by creating the idea of a submodule in the instrument class. This is just a class which implements the Instrument class, and may be a channel list or may just implement a logical collection of parameters within a parent instrument. For example, looking at the Lakeshore Model_336 driver, each channel can either be accessed as a member of a channel list, or by using the name of the channel: ```python3 therm = Model_336("therm", "...") # These two accesses are equivalent therm.A.temperature() therm.channels[0].temperature() ``` * style: remove force arg in snapshot_base Should think about the best way to implement this change. * style: remove unessecary import * fix: error in type check. * fix: submodules should be Metadatable Submodule objects must be Metadatables at minimum, instrument is too restrictive for channels. * Start adding tests for channels * Extend tests * Fix: ensure consistent dataset names regardless of chan[] or chanA reading * More work on channels testing * Make sure that instument is closed in tests * Depend on hypothesis for test * fix improve testing of channels * test more: * fix: channels pep8 * fix: ensure that channels can be extended from generator * More tests of adding channels * Add missing functions and parameters to channellist * fix: function testS * assert to unittest style * Revert "fix: function testS" This reverts commit ff8a316. * Revert "Add missing functions and parameters to channellist" This reverts commit 347c0b2. * Remove docs of non existing attributes * Docs: decadac quote *IDN to remove warning * Add some channel notebooks * fix: test_channels small tweeks * Add support to array parameters in channels And error clearly if you try to measure multiple multiparameters * Mock parameters add instruments to fix names in tests * refactor tests to reduce code duplication * Revert "Add some channel notebooks" This reverts commit 2dbacc0. * feat: Add channelised QDac driver Add a QDac driver using the new channelisation feature * feat: Add a set method to MultiChannelInstrumentParameter * docs: Add a driver example for the channelised Qdac * feat: Make ChannelList take default paramclass Make the paramclass returned by __getattr__ be customisable. * feat: Add a channelised QDac driver Add a channelised driver for the QDac. This requires changing the base code for channels a bit to allow for custom MultiChannelInstrumentParameter subclasses. * docs: Add notebook for QDac w. channels * style: Remove unnecessary comment * Fix syntax/spelling errors in DAC driver * Fix: qdac_channels remove magic number * Doc: clarify doc string * Fix: rename paramclass to be more specific * Lakeshore tab to space * pep8 lakeshore driver * Pep8 decadac driver * Fix warnings in qdac driver * Annotate parameters that linter has problems with
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce
Expected behaviour
should return the voltage that channel is at.
Actual behaviour
Returns 0.03... (Consistently the same number every time on all channels which is totally unrelated to actual output, would write whole number but am on my phone so will add it in comments when I get a chance).
Any ideas? Possibly it's converting a message and there is a missing read() somewhere. I didn't get the chance to play around but if anyone has a chance to take a quick look (or while testing #552 pr). Cheers.
@WilliamHPNielsen
The text was updated successfully, but these errors were encountered: