#1932: ML_BSSN: other_timelevels Parameter Not Respected
--------------------------------+-------------------------------------------
Reporter: zachetie@… | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Other | Version: development version
Keywords: |
--------------------------------+-------------------------------------------
At the Jun 27, 2016 ET telecon, we found the following bug in
McLachlan/ML_BSSN:
Inside the ET_2016_05 ML_BSSN/schedule.ccl, you'll notice the following
lines:
STORAGE: ML_Ham[timelevels]
STORAGE: ML_mom[timelevels]
STORAGE: ML_cons_detg[timelevels]
STORAGE: ML_cons_Gamma[timelevels]
STORAGE: ML_cons_traceA[timelevels]
in all of these lines, "timelevels" should be replaced by
"other_timelevels".
This should result in significantly increased memory usage in the latest
ML_BSSN (possibly at the 10-20% level), particularly in vacuum evolutions.
Related to this problem, I noticed that in a previous version of McLachlan
(2015_05, where the above issue does not exist), all constraints are being
stored in checkpoint files, despite having only one timelevel.
ML_BSSN_Helper is supposed to overwrite the ML_BSSN/interface.ccl request
to set the Checkpoint="no" tag.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1932>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1965: use CCTK_BOOLEAN for useSpatialBetaDriver in ML_BSSN
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: ML_BSSN |
-----------------------------------+----------------------------------------
This parameter used to be a string keyword accepting "yes" and "no" and
was changed to a CCTK_INT in the ML rewrite. Making it a CCTK_BOOLEAN
won't change it's C type (both CCTK_INT and CCTK_BOOLEAN are ints in C++)
but will let users use the same parfile with both pre and post-rewrite
code (or alternative the same code with pre and post-rewrite parfiles).
If we really cared about the speed we *may* consider making it a CCTK_REAL
so that {{{ifthen(v==1, a,b)}}} could be written as {{{ifthen(v, a,b)}}}
thus saving on the comparison operation required and on a int->real
conversion.
This would require adding support for Boolean Parameters to Kranc's
Param.m which does not seem difficult.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1965>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1969: Carpet::processor_topology = "recursive" does not work with one process
---------------------+------------------------------------------------------
Reporter: sbrandt | Owner:
Type: defect | Status: new
Priority: minor | Milestone:
Component: Other | Version: development version
Keywords: |
---------------------+------------------------------------------------------
If you set Carpet::processor_topology = "recursive" and use one mpi
process, you get the following error:
{{{
terminate called after throwing an instance of 'std::out_of_range'
what(): vector::_M_range_check: __n (which is 0) >= this->size() (which
is 0)
Rank 0 with PID 19719 received signal 6
Writing backtrace to ./backtrace.0.txt
Aborted (core dumped)
}}}
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1969>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1966: provide strerror() type function in Cactus
-------------------------+--------------------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: optional | Milestone:
Component: Cactus | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
It would be useful to have a {{{sterror()}}} like function in Cactus that
could turn the error codes returned by the various flesh functions into
user readable strings.
This requires mostly that we consolidate all error codes into a single
file to make sure there are no duplicates.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1966>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1961: Carpet treats prolongation_operator=none as prolongation_operator=sync
-------------------------+--------------------------------------------------
Reporter: rhaas | Owner: eschnett
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: Carpet | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
Currently Carpet treats a user set prolongation operator of "none" the
same as "sync" (see line 2187 of SetupGH.cc in the
{{{get_transport_operator}}} routine:
{{{
if (CCTK_Equals(prolong_string, "none")) {
// This would surprise too many people
// return op_none;
} else if (CCTK_Equals(prolong_string, "sync")) {
return op_sync;
}}}
It would however sometimes be useful to actually have a prolongation
operator that does nothing so that one can leave in the SYNC statements in
the schedule (which cannot be made conditional) and switch a variable from
being correctly computed everywhere to being correct only in the interior
of the components if this variable is eg only for diagnostics and one can
avoid inspecting the incorrect values at the borders.
I would thus like to introduce a new prolongation operator string
"no_synchronization" that should do what "none" claimed to do and would
internally actually be translated into {{{op_none}}}. Or alternative make
"none" map to "op_none" again but then this may surprise people once more.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1961>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1945: NsNsToHMNS gallery example: parfile defective
---------------------------------+------------------------------------------
Reporter: dradice@… | Owner:
Type: defect | Status: new
Priority: critical | Milestone:
Component: Other | Version:
Keywords: |
---------------------------------+------------------------------------------
There is a bug in the parfile of the NsNsToHMNS gallery example. I am
attaching a patch.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1945>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1963: The thorn NsTracker is not available
---------------------------+------------------------------------------------
Reporter: anonymous | Owner:
Type: defect | Status: new
Priority: unset | Milestone:
Component: GetComponents | Version:
Keywords: NsTracker |
---------------------------+------------------------------------------------
Using GetComponents with
http://einsteintoolkit.org/about/gallery/NsNsToHMNS/NsNsToHMNS.th gives an
error that NsTracker can not be downloaded.
It tries to fetch it from the branch directory, which is empty, so you
have to download it manually from the parent directory in which /src is
in.
After manual download it works!
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1963>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1960: ML_BSSN ues its timelevel parameter for the constraints rather than
other_timelevel
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: optional | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: ML_BSSN |
-----------------------------------+----------------------------------------
Looking at ML_BSSN's storage.ccl it seems as if it uses its timelevel
parameter to control how many time levels are allocated and used for the
constraints. However in ML_BSSN_Helper it seems clear (in
ML_BSSN_SetGroupTags) that originally other_timelevels (which also fits
better with the description of the parameter) should instead be used so
that one can use different numbers of time levels for the constraints (ie.
only 1 so that prolongation etc can be disabled which ML_BSSN_Helper does
if other_timelevels is set to 0, however the extra memory is still
allocated).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1960>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1957: add Justin Shive to simfactory committers list
------------------------+---------------------------------------------------
Reporter: rhaas | Owner:
Type: task | Status: new
Priority: major | Milestone:
Component: SimFactory | Version: development version
Keywords: |
------------------------+---------------------------------------------------
Justin Schive kindly agreed to maintain the simfactory entry for the UIUC
golub cluster that is currently under review in #1956. If possible, I
would like to ask the owner of the simfacory project on bitbucket to add
Justin (user name hyschive) to the project, please.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1957>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit