Skip to content

Commit

Permalink
Merge pull request #5496 from jenshnielsen/agilent_E8257D_typo
Browse files Browse the repository at this point in the history
fix call to incorrect parameter in Agilent8257D driver
  • Loading branch information
jenshnielsen authored Nov 9, 2023
2 parents af1110b + 1e2ea28 commit df77444
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions docs/changes/newsfragments/5496.improved
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fixed a bug in the Agilent 8257D where on and off would call a non existing parameter. The methods have been updated to call ``output_enabled``.
4 changes: 2 additions & 2 deletions src/qcodes/instrument_drivers/agilent/Agilent_E8257D.py
Original file line number Diff line number Diff line change
Expand Up @@ -132,10 +132,10 @@ def __init__(
self.connect_message()

def on(self) -> None:
self.set("status", "on")
self.output_enabled.set("on")

def off(self) -> None:
self.set("status", "off")
self.output_enabled.set("off")

# functions to convert between rad and deg
@staticmethod
Expand Down

0 comments on commit df77444

Please sign in to comment.