fix dubious if condition in .github/scripts/parse.py, add cryo libs

.github/scripts/parse_rpt.py

@@ -52,15 +52,9 @@
         else:
             print("LVS is clean!")
 
-if ((len(sys.argv) > 1) and ((sys.argv[1] == "sky130hvl_ldo") or (sys.argv[1] == "sky130hvl_ldo_full"))) or ((len(sys.argv) > 1) and (sys.argv[1] == "sky130hd_cryo")):
-    print("Flow check is clean!")
-else:
-    json_filename = "test.json"
+json_filename = "test.json"
 
-    if os.path.exists(json_filename):
-        if check_gen_files():
-            print("Flow check is clean!")
-        else:
-            print("Flow check failed!")
-    else:
-        raise ValueError(".json config file not found!")
+if check_gen_files(json_filename, (len(sys.argv) == 1)):
+        print("Flow check is clean!")
+else:
+    print("Flow check failed!")

openfasoc/generators/common/__init__.py

@@ -8,7 +8,7 @@
 - `common.simulation`
 	1. `run_simulations()`: Used to run SPICE testbenches with multiple parameters.
 - `common.check_gen_files`
-	1. `check_gen_files()`: Used to check if the various files that should be generated by the flow are present in their required directories. 
+	1. `check_gen_files(parameters: json_filename, is_tempsense)`: Used to check if the various files that should be generated by the flow are present in their required directories. 
 - `common.check_gen_extensions`
 	1. Stores the extensions of the files generated by the flow. 
 See individual function documentation for more information on a particular function.

openfasoc/generators/common/check_gen_files.py

@@ -1,8 +1,8 @@
 import json
 import os
 
-def check_gen_files():
-    with open('test.json', 'r') as file:
+def check_gen_files(json_filename, is_tempsense) -> int:
+    with open(json_filename) as file:
         data = json.load(file)
 
     # print('Found .json config file...')
@@ -33,10 +33,10 @@ def check_gen_files():
                     if (os.path.exists(file) == 0):
                         raise ValueError(file + " does not exist!")
     # print("Found necessary work result files!")
-
-    for file in ("error_within_x.csv", "golden_error_opt.csv", "search_result.csv"):
-        if os.path.exists(file) == 0:
-            raise ValueError(file + " does not exist!")
+    if (is_tempsense):
+        for file in ("error_within_x.csv", "golden_error_opt.csv", "search_result.csv"):
+            if os.path.exists(file) == 0:
+                raise ValueError(file + " does not exist!")
 
     #print("Found generated .csv files!")
     return 1

openfasoc/generators/cryo-gen/tools/parse_rpt.py

@@ -5,15 +5,18 @@
 sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..'))
 from common.check_gen_files import check_gen_files
 
-drc_filename = "flow/reports/sky130hd/cryo/6_final_drc.rpt"
+dir_path = r'flow/reports'
+lib = os.listdir(dir_path)
+
+drc_filename = "flow/reports/" + str(lib[0]) + "/cryo/6_final_drc.rpt"
 num_lines = sum(1 for line in open(drc_filename))
 
 if num_lines > 3:
     raise ValueError("DRC failed!")
 else:
     print("DRC is clean!")
 
-lvs_filename = "flow/reports/sky130hd/cryo/6_final_lvs.rpt"
+lvs_filename = "flow/reports/" + str(lib[0]) + "/cryo/6_final_lvs.rpt"
 lvs_line = subprocess.check_output(["tail", "-1", lvs_filename]).decode(
     sys.stdout.encoding
 )
@@ -28,10 +31,7 @@
 
 json_filename = "test.json"
 
-if os.path.exists(json_filename):
-    if check_gen_files():
+if check_gen_files(json_filename, (len(sys.argv) == 1)):
         print("Flow check is clean!")
-    else:
-        print("Flow check failed!")
 else:
-    raise ValueError(".json config file not found!")
+    print("Flow check failed!")

openfasoc/generators/ldo-gen/blocks/sky130hvl/ldo_place.txt

@@ -11,253 +11,3 @@
 {pt_array_unit\\[10\\]} {pt_array_unit\[10\]}
 {pt_array_unit\\[11\\]} {pt_array_unit\[11\]}
 {pt_array_unit\\[12\\]} {pt_array_unit\[12\]}
-{pt_array_unit\\[13\\]} {pt_array_unit\[13\]}
-{pt_array_unit\\[14\\]} {pt_array_unit\[14\]}
-{pt_array_unit\\[15\\]} {pt_array_unit\[15\]}
-{pt_array_unit\\[16\\]} {pt_array_unit\[16\]}
-{pt_array_unit\\[17\\]} {pt_array_unit\[17\]}
-{pt_array_unit\\[18\\]} {pt_array_unit\[18\]}
-{pt_array_unit\\[19\\]} {pt_array_unit\[19\]}
-{pt_array_unit\\[20\\]} {pt_array_unit\[20\]}
-{pt_array_unit\\[21\\]} {pt_array_unit\[21\]}
-{pt_array_unit\\[22\\]} {pt_array_unit\[22\]}
-{pt_array_unit\\[23\\]} {pt_array_unit\[23\]}
-{pt_array_unit\\[24\\]} {pt_array_unit\[24\]}
-{pt_array_unit\\[25\\]} {pt_array_unit\[25\]}
-{pt_array_unit\\[26\\]} {pt_array_unit\[26\]}
-{pt_array_unit\\[27\\]} {pt_array_unit\[27\]}
-{pt_array_unit\\[28\\]} {pt_array_unit\[28\]}
-{pt_array_unit\\[29\\]} {pt_array_unit\[29\]}
-{pt_array_unit\\[30\\]} {pt_array_unit\[30\]}
-{pt_array_unit\\[31\\]} {pt_array_unit\[31\]}
-{pt_array_unit\\[32\\]} {pt_array_unit\[32\]}
-{pt_array_unit\\[33\\]} {pt_array_unit\[33\]}
-{pt_array_unit\\[34\\]} {pt_array_unit\[34\]}
-{pt_array_unit\\[35\\]} {pt_array_unit\[35\]}
-{pt_array_unit\\[36\\]} {pt_array_unit\[36\]}
-{pt_array_unit\\[37\\]} {pt_array_unit\[37\]}
-{pt_array_unit\\[38\\]} {pt_array_unit\[38\]}
-{pt_array_unit\\[39\\]} {pt_array_unit\[39\]}
-{pt_array_unit\\[40\\]} {pt_array_unit\[40\]}
-{pt_array_unit\\[41\\]} {pt_array_unit\[41\]}
-{pt_array_unit\\[42\\]} {pt_array_unit\[42\]}
-{pt_array_unit\\[43\\]} {pt_array_unit\[43\]}
-{pt_array_unit\\[44\\]} {pt_array_unit\[44\]}
-{pt_array_unit\\[45\\]} {pt_array_unit\[45\]}
-{pt_array_unit\\[46\\]} {pt_array_unit\[46\]}
-{pt_array_unit\\[47\\]} {pt_array_unit\[47\]}
-{pt_array_unit\\[48\\]} {pt_array_unit\[48\]}
-{pt_array_unit\\[49\\]} {pt_array_unit\[49\]}
-{pt_array_unit\\[50\\]} {pt_array_unit\[50\]}
-{pt_array_unit\\[51\\]} {pt_array_unit\[51\]}
-{pt_array_unit\\[52\\]} {pt_array_unit\[52\]}
-{pt_array_unit\\[53\\]} {pt_array_unit\[53\]}
-{pt_array_unit\\[54\\]} {pt_array_unit\[54\]}
-{pt_array_unit\\[55\\]} {pt_array_unit\[55\]}
-{pt_array_unit\\[56\\]} {pt_array_unit\[56\]}
-{pt_array_unit\\[57\\]} {pt_array_unit\[57\]}
-{pt_array_unit\\[58\\]} {pt_array_unit\[58\]}
-{pt_array_unit\\[59\\]} {pt_array_unit\[59\]}
-{pt_array_unit\\[60\\]} {pt_array_unit\[60\]}
-{pt_array_unit\\[61\\]} {pt_array_unit\[61\]}
-{pt_array_unit\\[62\\]} {pt_array_unit\[62\]}
-{pt_array_unit\\[63\\]} {pt_array_unit\[63\]}
-{pt_array_unit\\[64\\]} {pt_array_unit\[64\]}
-{pt_array_unit\\[65\\]} {pt_array_unit\[65\]}
-{pt_array_unit\\[66\\]} {pt_array_unit\[66\]}
-{pt_array_unit\\[67\\]} {pt_array_unit\[67\]}
-{pt_array_unit\\[68\\]} {pt_array_unit\[68\]}
-{pt_array_unit\\[69\\]} {pt_array_unit\[69\]}
-{pt_array_unit\\[70\\]} {pt_array_unit\[70\]}
-{pt_array_unit\\[71\\]} {pt_array_unit\[71\]}
-{pt_array_unit\\[72\\]} {pt_array_unit\[72\]}
-{pt_array_unit\\[73\\]} {pt_array_unit\[73\]}
-{pt_array_unit\\[74\\]} {pt_array_unit\[74\]}
-{pt_array_unit\\[75\\]} {pt_array_unit\[75\]}
-{pt_array_unit\\[76\\]} {pt_array_unit\[76\]}
-{pt_array_unit\\[77\\]} {pt_array_unit\[77\]}
-{pt_array_unit\\[78\\]} {pt_array_unit\[78\]}
-{pt_array_unit\\[79\\]} {pt_array_unit\[79\]}
-{pt_array_unit\\[80\\]} {pt_array_unit\[80\]}
-{pt_array_unit\\[81\\]} {pt_array_unit\[81\]}
-{pt_array_unit\\[82\\]} {pt_array_unit\[82\]}
-{pt_array_unit\\[83\\]} {pt_array_unit\[83\]}
-{pt_array_unit\\[84\\]} {pt_array_unit\[84\]}
-{pt_array_unit\\[85\\]} {pt_array_unit\[85\]}
-{pt_array_unit\\[86\\]} {pt_array_unit\[86\]}
-{pt_array_unit\\[87\\]} {pt_array_unit\[87\]}
-{pt_array_unit\\[88\\]} {pt_array_unit\[88\]}
-{pt_array_unit\\[89\\]} {pt_array_unit\[89\]}
-{pt_array_unit\\[90\\]} {pt_array_unit\[90\]}
-{pt_array_unit\\[91\\]} {pt_array_unit\[91\]}
-{pt_array_unit\\[92\\]} {pt_array_unit\[92\]}
-{pt_array_unit\\[93\\]} {pt_array_unit\[93\]}
-{pt_array_unit\\[94\\]} {pt_array_unit\[94\]}
-{pt_array_unit\\[95\\]} {pt_array_unit\[95\]}
-{pt_array_unit\\[96\\]} {pt_array_unit\[96\]}
-{pt_array_unit\\[97\\]} {pt_array_unit\[97\]}
-{pt_array_unit\\[98\\]} {pt_array_unit\[98\]}
-{pt_array_unit\\[99\\]} {pt_array_unit\[99\]}
-{pt_array_unit\\[100\\]} {pt_array_unit\[100\]}
-{pt_array_unit\\[101\\]} {pt_array_unit\[101\]}
-{pt_array_unit\\[102\\]} {pt_array_unit\[102\]}
-{pt_array_unit\\[103\\]} {pt_array_unit\[103\]}
-{pt_array_unit\\[104\\]} {pt_array_unit\[104\]}
-{pt_array_unit\\[105\\]} {pt_array_unit\[105\]}
-{pt_array_unit\\[106\\]} {pt_array_unit\[106\]}
-{pt_array_unit\\[107\\]} {pt_array_unit\[107\]}
-{pt_array_unit\\[108\\]} {pt_array_unit\[108\]}
-{pt_array_unit\\[109\\]} {pt_array_unit\[109\]}
-{pt_array_unit\\[110\\]} {pt_array_unit\[110\]}
-{pt_array_unit\\[111\\]} {pt_array_unit\[111\]}
-{pt_array_unit\\[112\\]} {pt_array_unit\[112\]}
-{pt_array_unit\\[113\\]} {pt_array_unit\[113\]}
-{pt_array_unit\\[114\\]} {pt_array_unit\[114\]}
-{pt_array_unit\\[115\\]} {pt_array_unit\[115\]}
-{pt_array_unit\\[116\\]} {pt_array_unit\[116\]}
-{pt_array_unit\\[117\\]} {pt_array_unit\[117\]}
-{pt_array_unit\\[118\\]} {pt_array_unit\[118\]}
-{pt_array_unit\\[119\\]} {pt_array_unit\[119\]}
-{pt_array_unit\\[120\\]} {pt_array_unit\[120\]}
-{pt_array_unit\\[121\\]} {pt_array_unit\[121\]}
-{pt_array_unit\\[122\\]} {pt_array_unit\[122\]}
-{pt_array_unit\\[123\\]} {pt_array_unit\[123\]}
-{pt_array_unit\\[124\\]} {pt_array_unit\[124\]}
-{pt_array_unit\\[125\\]} {pt_array_unit\[125\]}
-{pt_array_unit\\[126\\]} {pt_array_unit\[126\]}
-{pt_array_unit\\[127\\]} {pt_array_unit\[127\]}
-{pt_array_unit\\[128\\]} {pt_array_unit\[128\]}
-{pt_array_unit\\[129\\]} {pt_array_unit\[129\]}
-{pt_array_unit\\[130\\]} {pt_array_unit\[130\]}
-{pt_array_unit\\[131\\]} {pt_array_unit\[131\]}
-{pt_array_unit\\[132\\]} {pt_array_unit\[132\]}
-{pt_array_unit\\[133\\]} {pt_array_unit\[133\]}
-{pt_array_unit\\[134\\]} {pt_array_unit\[134\]}
-{pt_array_unit\\[135\\]} {pt_array_unit\[135\]}
-{pt_array_unit\\[136\\]} {pt_array_unit\[136\]}
-{pt_array_unit\\[137\\]} {pt_array_unit\[137\]}
-{pt_array_unit\\[138\\]} {pt_array_unit\[138\]}
-{pt_array_unit\\[139\\]} {pt_array_unit\[139\]}
-{pt_array_unit\\[140\\]} {pt_array_unit\[140\]}
-{pt_array_unit\\[141\\]} {pt_array_unit\[141\]}
-{pt_array_unit\\[142\\]} {pt_array_unit\[142\]}
-{pt_array_unit\\[143\\]} {pt_array_unit\[143\]}
-{pt_array_unit\\[144\\]} {pt_array_unit\[144\]}
-{pt_array_unit\\[145\\]} {pt_array_unit\[145\]}
-{pt_array_unit\\[146\\]} {pt_array_unit\[146\]}
-{pt_array_unit\\[147\\]} {pt_array_unit\[147\]}
-{pt_array_unit\\[148\\]} {pt_array_unit\[148\]}
-{pt_array_unit\\[149\\]} {pt_array_unit\[149\]}
-{pt_array_unit\\[150\\]} {pt_array_unit\[150\]}
-{pt_array_unit\\[151\\]} {pt_array_unit\[151\]}
-{pt_array_unit\\[152\\]} {pt_array_unit\[152\]}
-{pt_array_unit\\[153\\]} {pt_array_unit\[153\]}
-{pt_array_unit\\[154\\]} {pt_array_unit\[154\]}
-{pt_array_unit\\[155\\]} {pt_array_unit\[155\]}
-{pt_array_unit\\[156\\]} {pt_array_unit\[156\]}
-{pt_array_unit\\[157\\]} {pt_array_unit\[157\]}
-{pt_array_unit\\[158\\]} {pt_array_unit\[158\]}
-{pt_array_unit\\[159\\]} {pt_array_unit\[159\]}
-{pt_array_unit\\[160\\]} {pt_array_unit\[160\]}
-{pt_array_unit\\[161\\]} {pt_array_unit\[161\]}
-{pt_array_unit\\[162\\]} {pt_array_unit\[162\]}
-{pt_array_unit\\[163\\]} {pt_array_unit\[163\]}
-{pt_array_unit\\[164\\]} {pt_array_unit\[164\]}
-{pt_array_unit\\[165\\]} {pt_array_unit\[165\]}
-{pt_array_unit\\[166\\]} {pt_array_unit\[166\]}
-{pt_array_unit\\[167\\]} {pt_array_unit\[167\]}
-{pt_array_unit\\[168\\]} {pt_array_unit\[168\]}
-{pt_array_unit\\[169\\]} {pt_array_unit\[169\]}
-{pt_array_unit\\[170\\]} {pt_array_unit\[170\]}
-{pt_array_unit\\[171\\]} {pt_array_unit\[171\]}
-{pt_array_unit\\[172\\]} {pt_array_unit\[172\]}
-{pt_array_unit\\[173\\]} {pt_array_unit\[173\]}
-{pt_array_unit\\[174\\]} {pt_array_unit\[174\]}
-{pt_array_unit\\[175\\]} {pt_array_unit\[175\]}
-{pt_array_unit\\[176\\]} {pt_array_unit\[176\]}
-{pt_array_unit\\[177\\]} {pt_array_unit\[177\]}
-{pt_array_unit\\[178\\]} {pt_array_unit\[178\]}
-{pt_array_unit\\[179\\]} {pt_array_unit\[179\]}
-{pt_array_unit\\[180\\]} {pt_array_unit\[180\]}
-{pt_array_unit\\[181\\]} {pt_array_unit\[181\]}
-{pt_array_unit\\[182\\]} {pt_array_unit\[182\]}
-{pt_array_unit\\[183\\]} {pt_array_unit\[183\]}
-{pt_array_unit\\[184\\]} {pt_array_unit\[184\]}
-{pt_array_unit\\[185\\]} {pt_array_unit\[185\]}
-{pt_array_unit\\[186\\]} {pt_array_unit\[186\]}
-{pt_array_unit\\[187\\]} {pt_array_unit\[187\]}
-{pt_array_unit\\[188\\]} {pt_array_unit\[188\]}
-{pt_array_unit\\[189\\]} {pt_array_unit\[189\]}
-{pt_array_unit\\[190\\]} {pt_array_unit\[190\]}
-{pt_array_unit\\[191\\]} {pt_array_unit\[191\]}
-{pt_array_unit\\[192\\]} {pt_array_unit\[192\]}
-{pt_array_unit\\[193\\]} {pt_array_unit\[193\]}
-{pt_array_unit\\[194\\]} {pt_array_unit\[194\]}
-{pt_array_unit\\[195\\]} {pt_array_unit\[195\]}
-{pt_array_unit\\[196\\]} {pt_array_unit\[196\]}
-{pt_array_unit\\[197\\]} {pt_array_unit\[197\]}
-{pt_array_unit\\[198\\]} {pt_array_unit\[198\]}
-{pt_array_unit\\[199\\]} {pt_array_unit\[199\]}
-{pt_array_unit\\[200\\]} {pt_array_unit\[200\]}
-{pt_array_unit\\[201\\]} {pt_array_unit\[201\]}
-{pt_array_unit\\[202\\]} {pt_array_unit\[202\]}
-{pt_array_unit\\[203\\]} {pt_array_unit\[203\]}
-{pt_array_unit\\[204\\]} {pt_array_unit\[204\]}
-{pt_array_unit\\[205\\]} {pt_array_unit\[205\]}
-{pt_array_unit\\[206\\]} {pt_array_unit\[206\]}
-{pt_array_unit\\[207\\]} {pt_array_unit\[207\]}
-{pt_array_unit\\[208\\]} {pt_array_unit\[208\]}
-{pt_array_unit\\[209\\]} {pt_array_unit\[209\]}
-{pt_array_unit\\[210\\]} {pt_array_unit\[210\]}
-{pt_array_unit\\[211\\]} {pt_array_unit\[211\]}
-{pt_array_unit\\[212\\]} {pt_array_unit\[212\]}
-{pt_array_unit\\[213\\]} {pt_array_unit\[213\]}
-{pt_array_unit\\[214\\]} {pt_array_unit\[214\]}
-{pt_array_unit\\[215\\]} {pt_array_unit\[215\]}
-{pt_array_unit\\[216\\]} {pt_array_unit\[216\]}
-{pt_array_unit\\[217\\]} {pt_array_unit\[217\]}
-{pt_array_unit\\[218\\]} {pt_array_unit\[218\]}
-{pt_array_unit\\[219\\]} {pt_array_unit\[219\]}
-{pt_array_unit\\[220\\]} {pt_array_unit\[220\]}
-{pt_array_unit\\[221\\]} {pt_array_unit\[221\]}
-{pt_array_unit\\[222\\]} {pt_array_unit\[222\]}
-{pt_array_unit\\[223\\]} {pt_array_unit\[223\]}
-{pt_array_unit\\[224\\]} {pt_array_unit\[224\]}
-{pt_array_unit\\[225\\]} {pt_array_unit\[225\]}
-{pt_array_unit\\[226\\]} {pt_array_unit\[226\]}
-{pt_array_unit\\[227\\]} {pt_array_unit\[227\]}
-{pt_array_unit\\[228\\]} {pt_array_unit\[228\]}
-{pt_array_unit\\[229\\]} {pt_array_unit\[229\]}
-{pt_array_unit\\[230\\]} {pt_array_unit\[230\]}
-{pt_array_unit\\[231\\]} {pt_array_unit\[231\]}
-{pt_array_unit\\[232\\]} {pt_array_unit\[232\]}
-{pt_array_unit\\[233\\]} {pt_array_unit\[233\]}
-{pt_array_unit\\[234\\]} {pt_array_unit\[234\]}
-{pt_array_unit\\[235\\]} {pt_array_unit\[235\]}
-{pt_array_unit\\[236\\]} {pt_array_unit\[236\]}
-{pt_array_unit\\[237\\]} {pt_array_unit\[237\]}
-{pt_array_unit\\[238\\]} {pt_array_unit\[238\]}
-{pt_array_unit\\[239\\]} {pt_array_unit\[239\]}
-{pt_array_unit\\[240\\]} {pt_array_unit\[240\]}
-{pt_array_unit\\[241\\]} {pt_array_unit\[241\]}
-{pt_array_unit\\[242\\]} {pt_array_unit\[242\]}
-{pt_array_unit\\[243\\]} {pt_array_unit\[243\]}
-{pt_array_unit\\[244\\]} {pt_array_unit\[244\]}
-{pt_array_unit\\[245\\]} {pt_array_unit\[245\]}
-{pt_array_unit\\[246\\]} {pt_array_unit\[246\]}
-{pt_array_unit\\[247\\]} {pt_array_unit\[247\]}
-{pt_array_unit\\[248\\]} {pt_array_unit\[248\]}
-{pt_array_unit\\[249\\]} {pt_array_unit\[249\]}
-{pt_array_unit\\[250\\]} {pt_array_unit\[250\]}
-{pt_array_unit\\[251\\]} {pt_array_unit\[251\]}
-{pt_array_unit\\[252\\]} {pt_array_unit\[252\]}
-{pt_array_unit\\[253\\]} {pt_array_unit\[253\]}
-{pt_array_unit\\[254\\]} {pt_array_unit\[254\]}
-{pt_array_unit\\[255\\]} {pt_array_unit\[255\]}
-{pt_array_unit\\[256\\]} {pt_array_unit\[256\]}
-{pt_array_unit\\[257\\]} {pt_array_unit\[257\]}
-{pt_array_unit\\[258\\]} {pt_array_unit\[258\]}
-{pt_array_unit\\[259\\]} {pt_array_unit\[259\]}
-{pt_array_unit\\[260\\]} {pt_array_unit\[260\]}
-{pt_array_unit\\[261\\]} {pt_array_unit\[261\]}
-{pt_array_unit\\[262\\]} {pt_array_unit\[262\]}

openfasoc/generators/temp-sense-gen/tools/parse_rpt.py

@@ -24,10 +24,7 @@
 
 json_filename = "test.json"
 
-if os.path.exists(json_filename):
-    if check_gen_files():
+if check_gen_files(json_filename, (len(sys.argv) == 1)):
         print("Flow check is clean!")
-    else:
-        print("Flow check failed!")
 else:
-    raise ValueError(".json config file not found!")
+    print("Flow check failed!")