Physical tests

Configs/Engine_cfg.yaml

@@ -19,8 +19,8 @@ DBInfo:
   use_database: true
 
 
-SerialCheckSafe: true
-UsingScanner: true
+SerialCheckSafe: false
+UsingScanner: false
 
 # TestHandler: {name: Local, remoteip: localhost}
 # TestHandler: {name: SSH, username: <username>, hostname: <hostname>, remoteip: <remoteip>}
@@ -33,47 +33,83 @@ PhysicalTest:
     desc_long: Check that the power and grounds are not shorted at the terminal, or
       between the inputs.
     desc_short: Measure resistance between power and ground
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
     required: 1
 
   - name: 1.5V Input Check
     desc_long: Check that resistance between across C906 or C908 is non-zero.
     desc_short: Check that the 1.5V input is not shorted.
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: 10V Input Check
     desc_long: Check that resistance between across C907 or C909 is non-zero.
     desc_short: Check that the 10V input is not shorted.
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: 1.2V Output Check
     desc_long: Check that resistance between across C904 or C904 or TP901 is non-zero.
     desc_short: Check that the 1.2V output is not shorted.
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: RX 2.5V Output Check 
     desc_long: Check that resistance across C902 is non-zero.
     desc_short: Check that the RX 2.5V output is not shorted.
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: TX 2.5V Output Check
     desc_long: Check that resistance across either C903 or TP902 is non-zero.
     desc_short: Check that the TX 2.5V output is not shorted.
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: LDO Output
     desc_long: Measure the votlage across either R911 or TP901 and verify that it is appropriate.
     desc_short: Check that the LDO output voltage is around 1.2V
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: LinPol RX Check
     desc_long: Check that voltages across either R905 or R902 is 2.5V.
     desc_short: Check that the RX voltage from the linppol is operating correctly
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
   - name: LinPol TX Check
     desc_long: Measure the voltage across either TP902 or R906 or C903 is 2.5V.
     desc_short: Check that the TX voltage from the linppol is operating correctly
     required: 1
+    data_fields: 
+        - name: Acceptable Value
+          critical: true
+          type: bool
 
 Test:
   - TestClass: TestXPWR
@@ -99,7 +135,7 @@ Test:
     TestScript: test_startup.py
     desc_long: Read lpgbt ids
     desc_short: Check ids
-    name: lpGBT ids
+    name: LPGBT ID
     required: 1
 
   - TestClass: TestlpGBTcom

PythonFiles/GUIConfig.py

@@ -39,8 +39,7 @@ def getNumPhysicalTest(self):
     # Returns the information necessary for physical test
     # Formatted as a dictionary
     def getPhysicalTestRequirements(self, num):
-        index = 0
-        for ptest in self.board_cfg["PhysicalTest"]:
+        for index, ptest in enumerate(self.board_cfg["PhysicalTest"]):
             if index == num:
                 return ptest
 

PythonFiles/GUIWindow.py

@@ -25,6 +25,7 @@
 from PythonFiles.Scenes.AddUserScene import AddUserScene
 from PythonFiles.Scenes.PostScanScene import PostScanScene
 from PythonFiles.Scenes.PhysicalScenes.Inspection1 import Inspection1
+from PythonFiles.Scenes.PhysicalScenes.GenericPhysicalScene import GenericPhysicalScene
 from PythonFiles.update_config import update_config
 import webbrowser
 
@@ -154,7 +155,7 @@ def create_test_frames(self, queue):
 
         # For the physical tests
         for test_idx,test in enumerate(physical_list):
-            self.test_frames.append(Inspection1(self, self.master_frame, self.data_holder, test_idx))
+            self.test_frames.append(GenericPhysicalScene(self, self.master_frame, self.data_holder, test_idx, test))
             self.test_frames[test_idx].grid(row=0, column=0)
             offset = offset + 1
 
@@ -411,6 +412,43 @@ def set_frame(self, _frame):
 
     #################################################
 
+    def critical_failure_popup(self):
+
+        logging.debug("GUIWindow: Critical test failed. Cannot proceed with testing")
+
+        self.popup = tk.Toplevel()
+        self.popup.title("Critical Failure") 
+        self.popup.geometry("300x150+500+300")
+        self.popup.grab_set()
+
+
+        frm_popup = tk.Frame(self.popup)
+        frm_popup.pack()
+
+        lbl_popup = tk.Label(
+            frm_popup, 
+            text = " This board has failed to pass\n a physical test. Cannot proceed.",
+            font = ('Arial', 13)
+            )
+        lbl_popup.grid(column = 0, row = 0, columnspan = 2, pady = 25)
+
+        def action():
+            self.destroy_popup()
+            self.reset_board()
+
+        btn_ok = tk.Button(
+            frm_popup,
+            width = 12,
+            height = 2,
+            text = "Exit",
+            font = ('Arial', 12),
+            relief = tk.RAISED,
+            command = lambda: action()
+        )
+        btn_ok.grid(column = 0, row = 1, columnspan=2)
+
+
+    #################################################
 
     def unable_to_exit(self):