Define circuit name at initialization #979
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I suggest to have the possibility to define the
name
of a circuit network during the circuit initialization, as argument of the constructor, e.g.,cir = rf.Circuit(cnx, name = 'circuit')
.To use a
network
inside acircuit
connection list requires thenetwork
to have aname
. Indeed in the__init__
ofcircuit.py
:Currently, one is forced to use a variable to store the circuit
network
, and then specify aname
, e.g.:I find the suggested option more convenient, and readable, especially when nesting several circuits inside other circuits.