#637: output puncture velocity in puncturetracker -----------------------------------+---------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Keywords: PunctureTracker | -----------------------------------+---------------------------------------- the attached patch allows the puncture velocities to be output (simply moves the result of an interpolation into grid functions).
#637: output puncture velocity in puncturetracker ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: review Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: PunctureTracker ------------------------------------+--------------------------------------- Changes (by rhaas):
* status: new => review
#637: output puncture velocity in puncturetracker ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: review Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: PunctureTracker ------------------------------------+---------------------------------------
Comment (by knarf):
The patch looks fine, but why do you define pt_vel_t if the only thing it will ever be set to is cctk_time?
#637: output puncture velocity in puncturetracker ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: fixed | Keywords: PunctureTracker ------------------------------------+--------------------------------------- Changes (by rhaas):
* status: review => closed * resolution: => fixed
Comment:
I only define pt_vel_t to do the very same thing that is done for pt_loc (maybe it also makes output nicer?). I applied the patch (as is, the extra double value should not hurt anyone and it makes pt_vel the same as pt_loc).
trac@lists.einsteintoolkit.org
-
Einstein Toolkit