- removed comment
TwoPunctures: output individual ADM masses even when tuning
Issue #150
closed
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 (6)
-
-
- 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
-
- changed status to closed
- edited description
- Log in to comment
- Assignee
- Type
- Priority
- Status
- closed
- Component
- Server Infrastructure
- Milestone
- ET_2011_05
- Version
- ET_2010_11
- Votes
- 0
- Watchers
- 0
I suggest to apply this patch.