#1729: Switch Carpet to new bboxset class implementation
-------------------------+--------------------------------------------------
Reporter: eschnett | Owner: eschnett
Type: enhancement | Status: new
Priority: major | Milestone:
Component: Carpet | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
Carpet has a new bboxset class implementation that scales to much larger
number of MPI processes. I suggest to make the new implementation the
default.
This is currently disabled, and can be enabled by adding the two flags
"-DCARPET_ENABLE_BBOXSET2 -DCARPET_USE_BBOXSET2" to CPPFLAGS.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1729>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1700: Use 64-bit integers when building PETSc
-----------------------------------+----------------------------------------
Reporter: eschnett | Owner:
Type: enhancement | Status: new
Priority: major | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: |
-----------------------------------+----------------------------------------
PETSc needs to be able to count up to the total (global) number of points
it uses, not just process-local points. 32-bit integers are not sufficient
for large runs. Note that e.g. Carpet already uses 64-bits integers for
the same reason.
I suggest to enable the respective option when building PETSc by default.
TATPETSc supports this, other thorns may need to be updated.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1700>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1727: Support CCTK_INT16
-------------------------+--------------------------------------------------
Reporter: eschnett | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: Cactus | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
I suggest to add support for 16-byte integers to Cactus. I have created
the respective patches and put them into "eschnett/int16" branches of the
flesh and various arrangements, including CactusBase, CactusPUGH, and
Carpet.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1727>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1694: try using fast fowards when accepting pull requests
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: optional | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: |
-----------------------------------+----------------------------------------
For small single change pull requests have a full git merge in the history
can be annoying since it clutter the history view (it uses up two lines,
it may show connecting both branches existing at the same time even if
there are no commits on master during that time).
We could try and instead find out if bitbucket can be made to allow fast
forwards in the pull request merge
(https://bitbucket.org/site/master/issue/6106/forced-non-fast-forward-
merge-of-pull). Currently apparently bitbucket uses --no-ff ie it
disallows fast forwards.
We could even think about forcing fast forwards only
(https://bitbucket.org/site/master/issue/9589/force-fast-forward-only-
merges-on-pull). This does however not work so well for big "feature"
branches that are brought back into master and where a fast forward may
fail and/or require extensive rebasing of the feature branch.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1694>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1663: VisIT CarpetHDF5 plugin pseudocolor error
---------------------------------------+------------------------------------
Reporter: bruno.giacomazzo@… | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Other | Version: development version
Keywords: visit |
---------------------------------------+------------------------------------
This problem has been present since the CarpetHDF5 plugin for VisIt became
part of the standard VisIt distribution and it is still present in the
current version of VisIt (2.8).
When Pseudocolor is used in log scale and Centering is set to Original or
Nodal, not all the values of the plotted quantity are shown (in particular
lower values are not plotted at all). This does not happen if one sets
Centering to Zonal, which instead shows the correct values.
I have attached a couple of images showing the problem (they show
hydrobase::rho for a BNS run).
I do not know what may be causing it, but it may cause serious errors when
analyzing data with VisIt (since one may miss some of the information on
the low value regions).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1663>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1635: hwloc requires a certain minimum version, but does not check for it
--------------------+-------------------------------------------------------
Reporter: knarf | Owner:
Type: defect | Status: new
Priority: major | Milestone: ET_2014_11
Component: Cactus | Version: development version
Keywords: |
--------------------+-------------------------------------------------------
Currently, hwloc's configuration.sh searched for any hwloc library and
uses it. However, it requires a pretty new version within some of its own
files, which leads to build failure on, e.g. Debian stable systems.
The attached patch uses pkg-config to get the version of the installed
hwloc library and if that version is older than 1.6 (educated guess, but
see
http://lists.einsteintoolkit.org/pipermail/users/2013-February/002860.html),
builds the bundled version even if another version is installed (but too
old).
Note that this is done (intentionally) only if no library was specified,
and the script was looking for it by itself. This allows users to specify
something which will not be overwritten by this mechanism.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1635>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1704: redirect stdout and stderr of all processes if requested (also of the root
process)
-------------------------+--------------------------------------------------
Reporter: knarf | Owner:
Type: enhancement | Status: new
Priority: optional | Milestone:
Component: Cactus | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
Currently Cactus only redirects, when asked, stdout and stderr to files
for processes other than the root process. The patch lets Cactus also
write stdout and stderr for the root process.
https://bitbucket.org/cactuscode/cactus/pull-request/2/redirect-stdout-
and-stderr-of-all/diff
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1704>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1702: AEILocalInterp and LocalInterp should output scheduled function name in
error messages
----------------------------------------------------------------+-----------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: AEILocalInterl LocalInterp CarpetInterp PUGHInterp |
----------------------------------------------------------------+-----------
the local interpolation routines abort if an interpolation cannot be
performed since the interpolation coordinates lie outside of the patch of
data they are given eg.
{{{
WARNING level 1 in thorn AEILocalInterp processor 0
(line 1109 of arrangements/AEIThorns/AEILocalInterp/src/Lagrange-tensor-
product/../template.c):
->
CCTK_InterpLocalUniform():
interpolation point is either outside the grid,
or inside but too close to the grid boundary!
(this may be caused by a global interpolation with
driver::ghost_size too small)
0-origin interpolation point number pt=54181 of
N_interp_points=54182
interpolation point (x,y,z)=(45.3992,15.5856,2.93915e-15)
grid x_min(delta_x)x_max = -0.198(0.066)1.584
grid y_min(delta_y)y_max = -0.198(0.066)1.518
grid z_min(delta_z)z_max = -0.198(0.066)1.452
}}}
However, if there are mutliple thorns that may call the interpolator it is
no always clear which caller was active when the error occured (eg it may
be the apparent horizon finder or the puncture tracker).
It would be nice if the interpolators were to report the currently
executing scheduled routine's name via CCTK_ScheduleQueryCurrentFunction.
That routine currently takes cctkGH as an argument but does not actually
use it. Thus, given the possible benefit, it would be acceptable I think
to call the routine with a NULL pointer (and a comment that this is
strictly not valid).
An alternative, which may be nicer, would be to have the driver add an
entry "cctkGH" to the table passed to {{{CCTK_InterpLocalUniform}}} in its
{{{param_table_handle}}}. The local interpolator can then savely use this
pointer if provided and (with an appropriate loud warning) fall back to
passing NULL if it is not found (and stop the fallback once the flesh
routine actually uses cctkGH).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1702>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1728: Remove CACHE_SIZE etc. from configure
-------------------------+--------------------------------------------------
Reporter: eschnett | Owner:
Type: enhancement | Status: new
Priority: optional | Milestone:
Component: Cactus | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
Configure cannot reliably detect CACHE_SIZE since this may be different on
build nodes and compute nodes. Thorn hwloc provides a reliable, portable
run-time solution. CACHE_SIZE is not currently used either. I suggest to
remove it from configure.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1728>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1724: Don't require configuration name in 'make CONFIG' if only one configuration
is present
-------------------------+--------------------------------------------------
Reporter: knarf | Owner: knarf
Type: enhancement | Status: new
Priority: optional | Milestone:
Component: Cactus | Version: development version
Keywords: |
-------------------------+--------------------------------------------------
For example 'make' after this pull request works if only one configuration
is present.
Otherwise a message would be output to tell the user to use 'make sim'
(for example) - which is kind of pointless because at that point make
"knew" already what was meant. It could, and should, just do it.
https://bitbucket.org/cactuscode/cactus/pull-request/6/dont-require-
configuration-name-when-only/diff
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1724>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit