[ET Trac] #6: Inconsistent computation of the volume form in Coordinates (llama) between Thornburg04/13 and default behavior

[Jordan Nicoules]

#6: Inconsistent computation of the volume form in Coordinates (llama) between Thornburg04/13 and default behavior

 Reporter: Jordan Nicoules
   Status: submitted
Milestone: 
  Version: 
     Type: bug
 Priority: major
Component: 

See ticket:
https://bitbucket.org/einsteintoolkit/tickets/issues/2912/inconsistent-computation-of-the-volume

**IMPORTANT WARNING: THIS IS A BEHAVIOR CHANGE AND MAY BREAK BACKWARD COMPATIBILITY FOR NON-THORNBURG04/13 RUNS!**

# Modifications to implement: #
## Coordinates/src/inverse_jacobian.F90 ##

See attached file.

### WARNING ###
This was tested only on Thornburg04 and Thornburg04nc. 
Thornburg13 works like Thornburg04 and redefines its volume form, so it should be fine. 
For other patch systems, I'm not sure if this is safe for each subpatch, but I think that shouldn't be more wrong than the current implementation.

### NOTE ###
Feel free to change the comments I made, and/or adapt the existing TODO in comment (lines 59-61) if it's relevant.

## Coordinates/src/thornburg04.cc

Fix typo in comment line 1439:
`// set volume form to determinant of Jacobian`

## Coordinates/src/thornburg13.cc

Fix typo in comment line 2237:
`// set volume form to determinant of Jacobian`
attachment: inverse_jacobian.F90 (https://api.bitbucket.org/2.0/repositories/llamacode/llama/issues/6/attachments/inverse_jacobian.F90)


--
Ticket URL: https://bitbucket.org/llamacode/llama/issues/6/inconsistent-computation-of-the-volume
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.einsteintoolkit.org/pipermail/trac/attachments/20260410/11d43d55/attachment.htm>