Skip to content
This repository has been archived by the owner on Jul 23, 2019. It is now read-only.

Issue with calculating contraction #14

Open
cjayross opened this issue Jul 8, 2019 · 1 comment · May be fixed by #15
Open

Issue with calculating contraction #14

cjayross opened this issue Jul 8, 2019 · 1 comment · May be fixed by #15

Comments

@cjayross
Copy link
Contributor

cjayross commented Jul 8, 2019 •
edited
Loading

For some reason the API has it that
equation

Currently, the left hand side yields the correct value while the right hand side sums the diagonal components of the array representing the tensor literally.

However this is only an issue with contractions on a single tensor (not a product of tensors), and when the contraction yields a scalar. Meaning that the calculation of the Ricci tensor is fine, but the Ricci scalar is not.
@cjayross
Copy link
Contributor Author

cjayross commented Jul 8, 2019 •
edited
Loading

This is an issue with https://github.com/sympy/sympy.

The ~sympy.tensor.tensor.Tensor._extract_data method contracts indices without applying the metric to the arrays in the replacement_dict argument.

@cjayross cjayross linked a pull request Jul 8, 2019 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ricci scalar fix
1 participant
@cjayross