Merge pull request #7344 from jjmartinezQT/Operation-modes-for-SGS100a

operation mode for SGS100A instrument

Author: astafan8

docs/changes/newsfragments/7344.improved_driver

@@ -0,0 +1 @@
+Added operation mode in the SGS100A instrument to be able to change between NORMal and BBBYpass operation modes. Defaults to NORMal.

docs/examples/driver_examples/Qcodes example with Rohde Schwarz SGS100A.ipynb

@@ -101,6 +101,16 @@
     "# stop RF outout\n",
     "sgsa.status(False)"
    ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "# set bypass operation mode\n",
+    "sgsa.operation_mode(\"BBBYpass\")"
+   ]
   }
  ],
  "metadata": {

src/qcodes/instrument/sims/RSSGS100A.yaml

@@ -201,6 +201,14 @@ devices:
         setter:
           q: "SOUR:IQ:IMP:QUAD {:.2f}"
 
+      operation_mode:
+        default: "NORMal"
+        getter:
+          q: ":SOUR:OPMode?"
+          r: "{}"
+        setter:
+          q: ":SOUR:OPMode {}"
+
 resources:
   GPIB::1::INSTR:
     device: SGS100A

src/qcodes/instrument_drivers/rohde_schwarz/SGS100A.py

@@ -322,6 +322,17 @@ def __init__(
             vals=vals.Numbers(20e-9, 100),
         )
         """Parameter pulsemod_width"""
+        self.operation_mode: Parameter = self.add_parameter(
+            "operation_mode",
+            label="Operation Mode",
+            get_cmd=":SOUR:OPMode?",
+            set_cmd=":SOUR:OPMode {}",
+            vals=vals.Enum(
+                "NORMal",  # Normal - Normal Mode
+                "BBBYpass",  # Bypass - Bypass Mode
+            ),
+        )
+        """Parameter operation_mode"""
         self.add_function("reset", call_cmd="*RST")
         self.add_function("run_self_tests", call_cmd="*TST?")
 

tests/drivers/test_RS_SGS100A.py

@@ -98,3 +98,8 @@ def test_IQ_gain_imbalance(sg) -> None:
 
 def test_IQ_angle(sg) -> None:
     verify_property(sg, "IQ_angle", [-8, -4, 0, 4, 8])
+
+
+def test_operation_mode(sg) -> None:
+    print(f"operation mode is set to {sg.operation_mode()}")
+    verify_property(sg, "operation_mode", ["NORMal", "BBBYpass"])