-
Notifications
You must be signed in to change notification settings - Fork 56
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
document similar_from_indices
?
#89
Comments
Yes, I changed how this works in the latest release but did not update the docs accordingly. Provided that your custom tensor type listens to the call |
Thanks for the quick response! What I am still wondering, is whether |
I think I mostly got it now, the only thing I'm still a bit confused about is the difference between the arguments |
That's something I specifically added for my use case in TensorKit.jl, where tensors have two sets of indices, and you can permute arbitrary among and in between them. For most cases, you should just join There is syntax in the left hand side to specify two types of indices, either as
or
For tensors in the right hand side, this is ignored and both sets of indices are anyway grouped into one. |
Ah, thanks, that makes sense! In CoolTensors, co- and contravariant indices can have an arbitrary order, so I don't think I will make use of this functionality, but that's good to know. Not at my PC atm, but I will try to put this into a docstring and make a PR. |
First of all, awesome package! I want to use TensorOperations in https://github.com/simeonschaub/CoolTensors.jl, but I think to get it to return the correct output type, I need to implement a specialized method of
similar_from_indices
. Would it be possible to add a short explanation on what the various arguments are exactly forsimilar_from_indices
?The text was updated successfully, but these errors were encountered: