Reformatting Layers #218
Reformatting Layers #218
Conversation
|
Quick Note on the most recent push: I made the entanglement folder separate and put the previous entanglement gates there along with the Op2 gates. |
01110011011101010110010001101111
marked this pull request as ready for review
| for _lyr in _all_layers: | ||
| # check the layer has a non-empty name | ||
| assert _lyr.name is not None, f"Layer name not defined for {layer}" | ||
| layer_name_dict[_lyr.name] = _lyr |
There was a problem hiding this comment.
Dynamically created the layer list here. Note that instead of specifying the dunder all (and therefore needing to rewrite everything again), I just used underscores to indicate not to export the variables.
Another thing to note is that I noticed the previous layer dictionary only had LayerTemplate0s (which is what’s included in the _all_layers). This dictionary, however, is only used in QuantumNode. Perhaps this can be reformatted for different purposes going forward.
There was a problem hiding this comment.
Also, for later reference, some discourse on using underscore versus all: https://stackoverflow.com/questions/44834/what-does-all-mean-in-python
Similar to previous PRs, reformatting the Layers into separate files.