#1875: SetMask_SphericalSurface: retain mask
------------------------------------+---------------------------------------
Reporter: cott@… | Owner:
Type: enhancement | Status: new
Priority: unset | Milestone:
Component: Other | Version: development version
Keywords: |
------------------------------------+---------------------------------------
SetMask_SphericalSurface sets a mask based on SphericalSurface
information. At every invocation, it wipes the mask, then resets it. Old
behavior: mask is set from surface only if surface is active and valid.
However, sometimes finding a horizon with AHFinderDirect may fail, but the
mask info (i.e. the radius) may be perfectly valid. Mask is still not set
in the old behavior, because AHFinderDirect makes the mask invalid (-1) if
it does not find a horizon. New behavior: set mask from surface is surface
is active and its minimum radius is > 0.
Pull request: https://bitbucket.org/einsteintoolkit/einsteinutils/pull-
requests/1/setmask_sphericalsurface-retain-mask/diff
Pull request also adds verbose output (default: off).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1875>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1390: parameter file parser aborts when findeing first error
--------------------+-------------------------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Cactus | Version: development version
Keywords: |
--------------------+-------------------------------------------------------
The attached parfile contains multiple errors (one per line).
However the parameter file parser only reports the first one, then stops.
This makes verifying parfiles for correctness hard. It might be good to
defer aborting until the end of the file or until a larger number of
parsing errors were encountered.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1390>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1172: Remove unnecessary exp/log calls in EOS_Omni
-----------------------------------+----------------------------------------
Reporter: eschnett | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: |
-----------------------------------+----------------------------------------
EOS_Omni seems to call exp/log more often than necessary in the nuc_eos
table lookup routines. Use algebraic identities to remove them.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1172>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#2003: piraha_everywhere Zelmani stress test
--------------------+-------------------------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Cactus | Version: development version
Keywords: piraha |
--------------------+-------------------------------------------------------
Hello all,
Zelmani contains a number of ccl files that make the perl based piraha
parser abort with errors but that parsed fine with the old parrser. Some
ccl files are clearly incorrect while others are correct and the parser
should accept them. The major such issue is that the parser requires that
that there is a private/public/protected setting at the beginning of a
param.ccl which is not required by the docs since parameters are supposed
to default to private.
I attach all ccl files both in their original version as well as version
that can be parsed without errors (hopefully).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/2003>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1916: GetComponents checks out git repositories serially
---------------------------+------------------------------------------------
Reporter: hinder | Owner:
Type: defect | Status: new
Priority: major | Milestone: ET_2016_11
Component: GetComponents | Version: development version
Keywords: |
---------------------------+------------------------------------------------
When called with --parallel, GetComponents still checks out git
repositories serially. This may have been introduced with #83, but there
is no information in that ticket about what was done.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1916>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#614: relative tolerence in test.ccl of QuasiLocalMeasures very high
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: optional | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: |
-----------------------------------+----------------------------------------
LSUThorns/QuasiLocalMeasures/test/test.ccl curerntly reads:
{{{
ABSTOL 1.e-7
RELTOL 1.e+5
}}}
I am curious: is the relative tolerance of 10,000 intentional or should it
have been 1e-5 instead?
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/614>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#2000: support radial boundary conidtion with Llama in NewRad
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: major | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: NewRad |
-----------------------------------+----------------------------------------
This implements the radially outgoing boundary condition on Llama's
spherical grid. I (Roland) checked that the expressions agree with what
CTGamma implements in its CTGRadiativeBC/src/calc_rhs.h, Ian checked that
the physics makes sense.
To use the this the user needs to set the parameter z_is_radial explicitly
rather than in CTGamma where CTGRadiativeBC decides this on its own based
on its knowledge of Llama's coordinate systems. If one really wanted to
risk being clever one could likely try and auto-detect this like so:
{{{
z_is_radial = (r[CCTK_GFINDEX3D(cctkGH, 0,0,1)] !=
r[CCTK_GFINDEX3D(cctkGH, 0,0,0)]) &&
(r[CCTK_GFINDEX3D(cctkGH, 0,1,0)] ==
r[CCTK_GFINDEX3D(cctkGH, 0,0,0)]) &&
(r[CCTK_GFINDEX3D(cctkGH, 1,0,0)] ==
r[CCTK_GFINDEX3D(cctkGH, 0,0,0)]);
}}}
though this is obviously not fully safe as it only tests at a single
point.
Pull request is here: https://bitbucket.org/einsteintoolkit/einsteinevolve
/pull-requests/9/newrad-add-simple-support-for-llama-outer/diff
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/2000>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1276: Intel 2013.1.117 mis-compiles NewRad
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: NewRad |
-----------------------------------+----------------------------------------
Intels compiler fails (with -O2) to push values for bmin onto the stack in
lines 316 of newrad.cc and line 126 of extrap.cc. Adding printf's for bmin
perturbs the bug out of existence, but adding a printf of the address of
bmax and reveals that at the time extrap_kernel is call the integer just
before this address is still the initialization value of bmin[2] and not
the correct value.
The attached patch disables optimization for the two driver functions
affected (but not the actual kernel).
The patch is specific (via an #if) for this particular compiler and
version. What is the best way of handling this? Target any intel version
starting from the known failing one until we know of known good one? Or
starting from an older known good one (that would be intel 11 in my case).
Hopefully no similar bug is triggered by Carpet's use of the same idiom.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1276>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1439: SSL certificate check failing
--------------------+-------------------------------------------------------
Reporter: rhaas | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: Other | Version: development version
Keywords: |
--------------------+-------------------------------------------------------
The check for SSL certificates in line 535:
{{{#perl
# check for svn SSL problems
if ( $rec{"TYPE"} eq "svn" && defined $rec{"AUTH_URL"} ) {
my $base = $rec{"AUTH_URL"};
$base =~ s/(https\:\/\/[\w\.]+)\/(.*)$/$1/i;
unless ( defined $svn_servers{$base} ) {
my $ret = `$svn --non-interactive info $rec{AUTH_URL} 2>&1`;
if ( $ret =~ /Server certificate verification failed/ ) {
$svn_servers{$base} = 0;
}
else {
$svn_servers{$base} = 1;
}
}
}
}}}
is incorrect since eg for the ET manifest where
{{{
AUTH_URL=https://svn.einsteintoolkit.org/$1/trunk
}}}
the executed svn command is:
{{{
svn --non-interactive info https://svn.einsteintoolkit.org/$1/trunk 2>&1
}}}
which actually returns and error:
{{{
svn: E175002: Unable to connect to a repository at URL
'https://svn.einsteintoolkit.org/trunk'
svn: E175002: The OPTIONS request returned invalid XML in the response:
XML parse error at line 1: Extra content at the end of the document
(https://svn.einsteintoolkit.org/trunk)
}}}
but the code does not test for svn failures at all at this point.
The simplest fix would be to move the check further down where {{{$1}}}
has been replaced by an actual value, eg into the loop:
{{{
# we are splitting each group of components into individuals
# to check for existence. they will now be passed individually to
# the checkout/update subroutines. this will take up more memory,
# but it should make it easier if the user decides to add another
# component from the same repository later
my @checkouts = split( /\s+/m, $rec{"CHECKOUT"} );
foreach my $checkout (@checkouts) {
}}}
in line 565 which however causes the test to run for every single CHECKOUT
item.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1439>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1985: GetCOmponents does not follow HTTP redirection when downloading via curl
---------------------------+------------------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: GetComponents | Version: development version
Keywords: |
---------------------------+------------------------------------------------
The pull request:
https://github.com/gridaphobe/CRL/pull/3
adds {{{--location}}} to curl calls to make curl follow redirects. wget
follows the redirect without further options.
The branch also contains two cleanup commits that remove unused code.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1985>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit