type checking in check_valid_params

Author: amzhao16

pygridsim/__pycache__/parameters.cpython-312.pyc

@@ -48,6 +48,8 @@ def check_valid_params(params, valid_params):
     for key in params:
         if key not in valid_params:
             raise KeyError(f"Parameter {key} is not supported")
+        if not isinstance(params[key], (int, float)):
+            raise TypeError("Parameter input should be int or float")
         if key in ["kV", "BasekV"] and params[key] < 0:
             raise ValueError("KV cannot be less than 0")
 

pygridsim/parameters.py

@@ -1,10 +1,10 @@
 {
     "Voltages": {
-        "source": 2275.42083457249,
-        "load0": 174.86504811289413
+        "source": 2077.428822659733,
+        "load0": 217.52249209519556
     },
     "Losses": {
-        "Active Power Loss": 331250.37737900193,
-        "Reactive Power Loss": 688674.6517717106
+        "Active Power Loss": 284917.58902059204,
+        "Reactive Power Loss": 592389.8824783975
     }
 }

sim.json

@@ -68,8 +68,8 @@ def test_003_one_source_one_load_exhaustive(self):
                     circuit.add_load_nodes(num=1, load_type=load_type.value)
                     circuit.add_lines([("source", "load0")], line_type.value)
                     circuit.solve()
-                    print("LineType:", line_type, "SourceType", source_type, "LoadType", load_type)
-                    print(circuit.results(["Voltages", "Losses"]))
+                    #print("LineType:", line_type, "SourceType", source_type, "LoadType", load_type)
+                    #print(circuit.results(["Voltages", "Losses"]))
                     circuit.clear()
 
 
@@ -251,6 +251,11 @@ def test_103_invalid_nodes_in_line(self):
         with self.assertRaises(ValueError):
             # only has source, load0 for now but tries to add another one
             circuit.add_lines([("source", "load5")])
+    
+    def test_104_non_int_parameters(self):
+        circuit = PyGridSim()
+        with self.assertRaises(TypeError):
+            circuit.add_load_nodes(params={"kV": "stringInput"})
 
 
 class TestLargeCircuit(unittest.TestCase):