- removed comment
TwoPunctures: output individual ADM masses even when tuning
Can we output the individual puncture ADM mass post-solve in TwoPunctures? At present (Chandrasekhar stable release), they're only quoted if we're tuning the mass through iterative solves (which, I suppose, is what most people do).
This requires only a handful of extra lines. I'm attaching an "svn diff" of my local version, which has this enabled.
Keyword:
Comments (5)
-
-
- removed comment
Why is PunctIntPolAtArbitPosition used, and not, as in the total ADM mass, PunctEvalAtArbitPosition? Otherwise the patch looks fine, despite the few unrelated changes.
-
reporter - removed comment
Replying to [comment:2 knarf]:
Why is PunctIntPolAtArbitPosition used, and not, as in the total ADM mass, PunctEvalAtArbitPosition? Otherwise the patch looks fine, despite the few unrelated changes.
Good question: I hadn't noticed the distinction, as I copied the new lines from what's in the earlier Newton-search loop. Perhaps they should -all- be PunctEvalAtArbitPosition (I assume the speed difference is negligible when evaluating at only two points)?
-
- changed status to open
- assigned issue to
- removed comment
-
- changed status to resolved
- removed comment
- Log in to comment
I suggest to apply this patch.