Implement Riemann symmetries in WeylScal4

Issue #1036 closed

Erik Schnetter created an issue 2012-08-09

WeylScal4 calculates 81 instead of 21 (or 20) components of the Riemann tensor. This makes code and run time about four times larger than necessary.

Keyword:

Comments (6)

Erik Schnetter reporter
- changed status to resolved
- removed comment
It turns out that the method suggested above does not work, as this applies the symmetries also to the shorthands, leading to zeros and minus signs in the shorthands.

I believe this will require a Kranc modification instead.
- 2012-08-10T11:26:11+00:00
Ian Hinder
- changed status to open
- removed comment
Kranc has been modified to eliminate the zeroes and minus signs (https://github.com/ianhinder/Kranc/commit/4707e0c3f03cc52d82f88735e602c52cf74f0cbe). The attached patch implements the tensor symmetries for R and R4p. I haven't applied any symmetries to Rojo. Resulting code passes tests. OK to apply (with the regenerate)?
- 2012-08-11T12:13:14+00:00
Ian Hinder
- changed status to open
- removed comment
- 2012-08-11T12:18:14+00:00
Barry Wardell
- changed status to open
- removed comment
This looks fine to me. Please apply.
- 2012-08-11T12:21:00+00:00
Ian Hinder
- changed status to resolved
- removed comment
Thanks. Committed as WeylScal4 r122.
- 2012-08-11T12:25:43+00:00
Roland Haas
- changed status to closed
- edited description
- 2019-02-21T20:11:04+00:00
Log in to comment