Repository navigation
Modified inductance circuit element implemented incorrectly #312
Description
The circuit element modified inductance (impedance.models.circuits.elements.La(p, f)) is stated correctly in the documentation, but implemented incorrectly in the code.
Currently it is implemented as Z = (L * 1j * omega) ** alpha. However it must be Z = L * (1j * omega) ** alpha.
Activity
Correction of La element acording to ECSHackWeek#312 and ECSHackWeek#286
shaurya20335 commentedon Sep 24, 2026
I'm a master's student at NYU Tandon School of Engineering and opened #337 to fix this. The reference values for
Laintest_each_elementwere computed with the same buggy formula, so I updated those as well, added a test that Z scales linearly with L, and changed the units of L toH sec^(a - 1)to match the documented definition.