Eliminate @init_in_parent (#431)

Moves the `__init__` hook into a metaclass hook on `__init__` instead of
the previously explicit `@init_in_parent` decorator. Cleans up the hook
logic. Deprecates `@init_in_parent`.

It seems this must be an `__init__` hook of some kind, since it needs to
inspect each `super().__init__` call, recursively, with the context it
is called in. The `__init__` hook does double duty:
- For the top-level block being elaborated, it materializes missing
params with dummy values
- For inner blocks, it 'wraps' arg values by creating a param based on
the type annotation

Internally, removes the `Block._init_param_values`, which used to be the
source of truth for init values. Instead, init params have an
InitParamBinding, which stores the value, if any

This changes (fixes, imo) some behavior:
- No longer materializes missing params except for the top-level block.
Calls to blocks that are missing params will now fail, as expected.
- *Expr (eg, RangeExpr, FloatExpr) are no longer allowed as annotation
types, instead must use *Like (eg, RangeLike, FloatLike)

Removes `@init_in_parent` in all the library code. There's a bunch.
Fixes some libraries which are broken. Removes obsoleted unit tests.

Adds a dead-simple LED test, probably the simplest possible circuit
here.

Resolves #409

Author: ducky64

edg/BoardTop.py

@@ -3,7 +3,6 @@
 
 class BaseBoardTop(DesignTop):
   """Design top with refinements for intermediate-level (0603+ SMD), hand-solderable components."""
-  @init_in_parent
   def __init__(self):
     super().__init__()
     self.refdes_prefix = self.Parameter(StringExpr())

edg/abstract_parts/AbstractAnalogSwitch.py

@@ -37,7 +37,6 @@ def __init__(self) -> None:
 class AnalogSwitchTree(AnalogSwitch, GeneratorBlock):
   """Generates an n-ported analog switch by creating a tree of individual, smaller switches.
   Parameterized by the size of the element switches."""
-  @init_in_parent
   def __init__(self, switch_size: IntLike = 0):
     super().__init__()
     self.switch_size = self.ArgParameter(switch_size)

edg/abstract_parts/AbstractAntenna.py

@@ -6,7 +6,6 @@
 
 @abstract_block
 class Antenna(Interface, Block):
-  @init_in_parent
   def __init__(self, frequency: RangeLike, impedance: RangeLike = Range.all(), power: RangeLike = (0, 0)*Watt):
     super().__init__()
 
@@ -31,7 +30,6 @@ class TableAntenna(Antenna, PartsTableSelector, GeneratorBlock):
   IMPEDANCE = PartsTableColumn(Range)
   POWER_RATING = PartsTableColumn(Range)
 
-  @init_in_parent
   def __init__(self, *args, **kwargs):
     super().__init__(*args, **kwargs)
     self.generator_param(self.frequency, self.power, self.impedance)

edg/abstract_parts/AbstractBjt.py

@@ -48,7 +48,6 @@ def Npn(*args, **kwargs) -> 'Bjt':
   def Pnp(*args, **kwargs) -> 'Bjt':
     return Bjt(*args, **kwargs, channel='PNP')
 
-  @init_in_parent
   def __init__(self, collector_voltage: RangeLike, collector_current: RangeLike, *,
                gain: RangeLike = Range.all(), power: RangeLike = Range.exact(0),
                channel: StringLike = StringExpr()) -> None:
@@ -91,7 +90,6 @@ class TableBjt(PartsTableSelector, Bjt):
   POWER_RATING = PartsTableColumn(Range)
   CHANNEL = PartsTableColumn(str)  # either 'PNP' or 'NPN'
 
-  @init_in_parent
   def __init__(self, *args, **kwargs):
     super().__init__(*args, **kwargs)
     self.generator_param(self.collector_voltage, self.collector_current, self.gain, self.power, self.channel)

edg/abstract_parts/AbstractCapacitor.py

@@ -85,7 +85,6 @@ class CapacitorStandardFootprint(StandardFootprint['Capacitor']):
 @abstract_block
 class UnpolarizedCapacitor(PassiveComponent):
   """Base type for a capacitor, that defines its parameters and without ports (since capacitors can be polarized)"""
-  @init_in_parent
   def __init__(self, capacitance: RangeLike, voltage: RangeLike, *,
                voltage_rating_derating: FloatLike = 0.5,
                exact_capacitance: BoolLike = False) -> None:
@@ -150,7 +149,6 @@ def parse_capacitor(cls, value: str) -> Tuple[Range, Range]:
   def block_from_symbol(cls, symbol_name: str, properties: Mapping[str, str]) -> 'Capacitor':
     return Capacitor(*cls.parse_capacitor(properties['Value']))
 
-  @init_in_parent
   def __init__(self, *args, **kwargs) -> None:
     super().__init__(*args, **kwargs)
 
@@ -178,7 +176,6 @@ class TableCapacitor(PartsTableSelector, Capacitor):
   NOMINAL_CAPACITANCE = PartsTableColumn(float)  # nominal capacitance, even with asymmetrical tolerances
   VOLTAGE_RATING = PartsTableColumn(Range)
 
-  @init_in_parent
   def __init__(self, *args, **kwargs):
     super().__init__(*args, **kwargs)
     self.generator_param(self.capacitance, self.voltage, self.voltage_rating_derating, self.exact_capacitance)
@@ -220,7 +217,6 @@ class TableDeratingCapacitor(TableCapacitor):
   DERATE_LOWEST = 0.2  # floor for maximum derating factor
   # LOOSELY approximated from https://www.maximintegrated.com/en/design/technical-documents/tutorials/5/5527.html
 
-  @init_in_parent
   def __init__(self, *args, single_nominal_capacitance: RangeLike = (0, 22)*uFarad, **kwargs):
     super().__init__(*args, **kwargs)
     self.single_nominal_capacitance = self.ArgParameter(single_nominal_capacitance)
@@ -292,7 +288,6 @@ class DummyCapacitorFootprint(DummyDevice, Capacitor, FootprintBlock):
 
   TODO: use footprint table?
   """
-  @init_in_parent
   def __init__(self, footprint: StringLike = "", manufacturer: StringLike = "", part_number: StringLike = "",
                value: StringLike = "",
                *args, **kwargs):
@@ -316,7 +311,6 @@ def symbol_pinning(self, symbol_name: str) -> Dict[str, BasePort]:
     assert symbol_name in ('Device:C', 'Device:C_Small', 'Device:C_Polarized', 'Device:C_Polarized_Small')
     return {'1': self.pwr, '2': self.gnd}
 
-  @init_in_parent
   def __init__(self, capacitance: RangeLike, *, exact_capacitance: BoolLike = False) -> None:
     super().__init__()
 
@@ -342,7 +336,7 @@ def connected(self, gnd: Optional[Port[GroundLink]] = None, pwr: Optional[Port[V
     return self
 
 
-class CombinedCapacitorElement(Capacitor):  # to avoid an abstract part error
+class CombinedCapacitorElement(Capacitor):
   def contents(self):
     super().contents()
     self.assign(self.actual_capacitance, self.capacitance)  # fake it, since a combined capacitance is handwavey
@@ -351,11 +345,11 @@ def contents(self):
 class CombinedCapacitor(MultipackDevice, MultipackBlock, GeneratorBlock):
   """A packed capacitor that combines multiple individual capacitors into a single component,
   with the sum of or taking the max of the constituent capacitances."""
-  @init_in_parent
   def __init__(self, *, extend_upper: BoolLike = False) -> None:
     super().__init__()
 
-    self.elements = self.PackedPart(PackedBlockArray(CombinedCapacitorElement()))
+    self.elements = self.PackedPart(PackedBlockArray(CombinedCapacitorElement(capacitance=RangeExpr(),
+                                                                              voltage=RangeExpr())))
     self.pos = self.PackedExport(self.elements.ports_array(lambda x: x.pos))
     self.neg = self.PackedExport(self.elements.ports_array(lambda x: x.neg))
     self.capacitances = self.PackedParameter(self.elements.params_array(lambda x: x.capacitance))

edg/abstract_parts/AbstractComparator.py

@@ -14,7 +14,6 @@ def symbol_pinning(self, symbol_name: str) -> Mapping[str, BasePort]:
     def block_from_symbol(cls, symbol_name: str, properties: Mapping[str, str]) -> 'Comparator':
         return Comparator()
 
-    @init_in_parent
     def __init__(self) -> None:
         super().__init__()
 

edg/abstract_parts/AbstractConnector.py

@@ -28,7 +28,6 @@ def __init__(self) -> None:
 
 class RfConnectorTestPoint(BlockInterfaceMixin[RfConnector]):
   """Test point mixin that allows the footprint to take a name"""
-  @init_in_parent
   def __init__(self, name: StringLike):
     super().__init__()
     self.tp_name = self.ArgParameter(name)

edg/abstract_parts/AbstractCrystal.py

@@ -33,7 +33,6 @@ class CrystalStandardFootprint(StandardFootprint['Crystal']):
 class Crystal(DiscreteComponent, HasStandardFootprint):
   _STANDARD_FOOTPRINT = CrystalStandardFootprint
 
-  @init_in_parent
   def __init__(self, frequency: RangeLike) -> None:
     """Discrete crystal component."""
     super().__init__()
@@ -60,7 +59,6 @@ class TableCrystal(PartsTableSelector, Crystal):
   FREQUENCY = PartsTableColumn(Range)
   CAPACITANCE = PartsTableColumn(float)
 
-  @init_in_parent
   def __init__(self, *args, **kwargs) -> None:
     """Discrete crystal component."""
     super().__init__(*args, **kwargs)
@@ -78,7 +76,6 @@ def _row_generate(self, row: PartsTableRow) -> None:
 
 @abstract_block_default(lambda: OscillatorCrystal)
 class OscillatorReference(DiscreteApplication):
-  @init_in_parent
   def __init__(self, frequency: RangeLike) -> None:
     """Crystal and supporting circuitry to connect it to an oscillator driver.
     Should include load capacitors."""

edg/abstract_parts/AbstractDevices.py

@@ -4,7 +4,6 @@
 
 @abstract_block
 class Battery(PowerSource):
-  @init_in_parent
   def __init__(self, voltage: RangeLike,
                current: RangeLike = RangeExpr.ZERO, *,
                capacity: FloatLike = 0.0):

edg/abstract_parts/AbstractDiodes.py

@@ -41,7 +41,6 @@ class BaseDiode(DiscreteSemiconductor, HasStandardFootprint):
   """
   _STANDARD_FOOTPRINT = DiodeStandardFootprint
 
-  @init_in_parent
   def __init__(self) -> None:
     super().__init__()
 
@@ -59,7 +58,6 @@ def symbol_pinning(self, symbol_name: str) -> Dict[str, BasePort]:
     assert symbol_name in ('Device:D', 'Device:D_Small')
     return {'A': self.anode, 'K': self.cathode}
 
-  @init_in_parent
   def __init__(self, reverse_voltage: RangeLike, current: RangeLike, *,
                voltage_drop: RangeLike = Range.all(),
                reverse_recovery_time: RangeLike = Range.all()) -> None:
@@ -95,7 +93,6 @@ class TableDiode(PartsTableSelector, Diode):
   FORWARD_VOLTAGE = PartsTableColumn(Range)  # possible forward voltage range
   REVERSE_RECOVERY = PartsTableColumn(Range)  # possible reverse recovery time
 
-  @init_in_parent
   def __init__(self, *args, **kwargs):
     super().__init__(*args, **kwargs)
     self.generator_param(self.reverse_voltage, self.current, self.voltage_drop, self.reverse_recovery_time)
@@ -125,7 +122,6 @@ def symbol_pinning(self, symbol_name: str) -> Dict[str, BasePort]:
     assert symbol_name in ('Device:D_Zener', 'Device:D_Zener_Small')
     return {'A': self.anode, 'K': self.cathode}
 
-  @init_in_parent
   def __init__(self, zener_voltage: RangeLike) -> None:
     super().__init__()
 
@@ -149,7 +145,6 @@ class TableZenerDiode(PartsTableSelector, ZenerDiode):
   ZENER_VOLTAGE = PartsTableColumn(Range)
   POWER_RATING = PartsTableColumn(Range)  # tolerable power
 
-  @init_in_parent
   def __init__(self, *args, **kwargs):
     super().__init__(*args, **kwargs)
     self.generator_param(self.zener_voltage)
@@ -167,7 +162,6 @@ def _row_generate(self, row: PartsTableRow) -> None:
 class ProtectionZenerDiode(Protection):
   """Zener diode reversed across a power rail to provide transient overvoltage protection (and become an incandescent
   indicator on a reverse voltage)"""
-  @init_in_parent
   def __init__(self, voltage: RangeLike):
     super().__init__()
 
@@ -188,7 +182,6 @@ def contents(self):
 @deprecated("Use AnalogClampResistor, which should be cheaper and cause less signal distortion")
 class AnalogClampZenerDiode(Protection, KiCadImportableBlock):
   """Analog overvoltage protection diode to clamp the input voltage"""
-  @init_in_parent
   def __init__(self, voltage: RangeLike):
     super().__init__()