Unified Pair syntax for sublats kwarg in hopping #29
+93
−58
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.
Addresses one of the points in #19
This simplifies and extends the
sublats
keyword inhopping
. A hoppings from sublat:A
to sublat:B
is now always denoted with:A => :B
. The PR also allows for broadcasting between two groups of sublattices(:A, :B) .=> (:A, :B)
(equivalent to(:A => :A, :B => :B)
), and even direct product,(:A, :B) => (:C, :D)
(equivalent to(:A=>:C,:A=>:D,:B=>:C,:B=>:D)
).Updated and extended also the docstrings for
hopping
andonsite
, and added an internal methodQuantica.sublats(model)
that returns the sublats of each term inmodel
.