- changed status to resolved
- removed comment
Implement Riemann symmetries in WeylScal4
Issue #1036
closed
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.
https://github.com/ianhinder/Kranc/issues/81 describes what to do.
Keyword:
Comments (6)
-
reporter -
- 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)?
-
- changed status to open
- removed comment
-
- changed status to open
- removed comment
This looks fine to me. Please apply.
-
- changed status to resolved
- removed comment
Thanks. Committed as WeylScal4 r122.
-
- changed status to closed
- edited description
- Log in to 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.