Skip to content
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

Style: Indices used in einsum calls #18

Open
alilasemi opened this issue Jul 30, 2021 · 0 comments
Open

Style: Indices used in einsum calls #18

alilasemi opened this issue Jul 30, 2021 · 0 comments
Labels
style Doesn't seem to be the best implementation

Comments

@alilasemi
Copy link
Collaborator

There is some level of standardization in the documentation about which letters to use to represent which indices in einsum calls (for example, i for elements, j for quadrature points), but duplicate indices are needed (currently just n for bases but we need probably 3, so maybe m, n, p?). Once a standard is set, then we need to go through all einsums and make sure they use the right letter.

@bornhoft bornhoft added the style Doesn't seem to be the best implementation label Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style Doesn't seem to be the best implementation
Projects
None yet
Development

No branches or pull requests

2 participants