-
Notifications
You must be signed in to change notification settings - Fork 19
IndsNetwork
additions
#9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Conflicts are resolved and tests pass locally, no idea how to fix the actual CI though. |
In #13 I deleted the file |
(I think I'll just have to bite the bullet and register all of the dependent packages to get CI working better.) |
My bad, removed it again. |
Thanks again for the contribution! |
Some changes and additions for
IndsNetwork
s:is_directed
forAbstractIndsNetwork
subtypes so they work withGraphs.SimpleTraits
uniqueinds
method similar to the one forAbstractITensorNetwork
.merge
method for mergingAbstractIndsNetwork
s, which just corresponds to a set union of all site and link indices of the network. Proved quite useful for me, though naming and usefulness in general might be debatable.IndsNetwork
constructor to allow specifying the indices for every site/link of the network individually by means of aDictionary
, as well as specifying multiple indices per site/link.siteind
system forIndsNetwork
s to allow passing either anInteger
,String
orFunction
sitetype, as well as specifying the sitetype for every site individually. Now also allows passing anyGraphs.AbstractGraph
subtype, solving Allow passing aGraphs.SimpleGraph
tositeinds
#7.