Open eliweissler opened 2 months ago
Hi @eliweissler,
Thank you for your interest in SQcircuit and its paper. I really appreciate your feedback. I believe your suggestion could solve the problem. I will take a deeper look at the circuit as soon as possible and update you accordingly.
In trying to analyze this circuit, I get the "The edge list does not specify a connected graph or all inductive loops of the circuit are not specified" error. As far as I can tell, both of those things are true:
Code to reproduce the error:
I think the error stems from the matrix inversion on line 395 of circuit.py. Requiring X to be square requires the number of inductive elements in the circuit to be equal to the number of non-ground nodes plus the number of loops. I think the "if" statement on line 380 catches all cases where this isn't true for 2 & 3 node circuits, but it misses some like this that are 4 nodes.
Could a possible fix could be to extend the "if" statement on line 380 to remove as many columns from K1 as is necessary to end up with a square X? I'm not sure if this would have some unexpected weirdness with assigning fluxes.
Thank you for your time and for developing SQcircuit! I've really enjoyed using the package and reading about the methods behind it!