cleaning

Author: ducky64

edg/core/Range.py

@@ -152,7 +152,8 @@ def intersects(self, other: 'Range') -> bool:
 
   def intersect(self, other: 'Range') -> 'Range':
     # TODO make behavior more consistent w/ compiler and returning empty that props as a unit
-    assert self.intersects(other), "Cannot intersect ranges that do not intersect"
+    if not self.intersects(other):
+      raise ValueError("cannot intersect ranges that do not intersect")
     return Range(max(self.lower, other.lower), min(self.upper, other.upper))
 
   def __add__(self, other: Union['Range', float]) -> 'Range':

examples/test_fcml.py

@@ -174,7 +174,8 @@ def generate(self):
 
 class FcmlPowerPath(InternalSubcircuit, GeneratorBlock):
   """FCML power path that accounts for inductor scaling behavior
-  TODO: this basically completely duplicates BuckConverterPowerPath, but adds a scaling factor that doesn't exist there
+  TODO: Is there a way to unify this with BuckConverterPowerPath?
+  This basically completely duplicates it, but adds a scaling factor that doesn't exist there
   """
   @init_in_parent
   def __init__(self, input_voltage: RangeLike, output_voltage: RangeLike, frequency: RangeLike,