#737: simfactory sync can lead to inconsistent builds
------------------------+---------------------------------------------------
Reporter: rhaas | Owner: eschnett
Type: defect | Status: new
Priority: major | Milestone:
Component: SimFactory | Version:
Keywords: |
------------------------+---------------------------------------------------
simfactory's sync command uses rsync with the --archive (-a) option. This
option set the modification date of files on the target machine to that on
the source machine. This can lead to inconsistent builds in the following
situation:
[1] local: edit source_file.c
[2] local: simfactory sync
[3] local: edit source_file.c
[4] remote: simfactory build
[5] local: simfactory sync
[6] remote: simfactory build
[6] will not rebuild source_file.o since [5] set the modification time to
that of [3] which which is older than [4] hence make will not consider
source_file.o to be out-of-date.
A simple fix is to add --no-times to the rsync options. Unfortunately this
disables rsync's file modification time optimization and requires it to
compute checksums on all files which can be very slow on slow filesystems.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/737>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1070: EOS_Omni::poly_gamma_ini should default to poly_gamma
-----------------------------------+----------------------------------------
Reporter: knarf | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version: development version
Keywords: |
-----------------------------------+----------------------------------------
At the moment, EOS_Omni::poly_gamma and EOS_Omni::poly_gamma_ini are two
parameters for the adiabatic exponent, one used only for initial data, the
other for evolutions. Both default to 2.0. This means that if someone,
unaware of poly_gamma_ini, changes only poly_gamma the result is usually
catastrophic (wrong results or c2p failures). I myself didn't notice today
and had to debug my way down to find my mistake.
I propose to make the default of EOS_Omni::poly_gamma_ini to be whatever
EOS_Omni::poly_gamma is. One way to achieve that would be to use some
special value for EOS_Omni::poly_gamma_ini which would usually not being
used to indicate that behavior and to make that default. However, this
being an exponent, at least in theory, all real values are allowed.
Another approach would be to introduce a new boolean parameter which would
allow users to use an initial value for gamma which is different from the
one of the initial data. This would be 'saver', but has the drawback of a
new parameter. Nevertheless, I favor the second approach.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1070>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1099: CarpetLib::interpolate_from_buffer_zones should by default set to "no"
--------------------+-------------------------------------------------------
Reporter: knarf | Owner: eschnett
Type: defect | Status: new
Priority: major | Milestone: ET_2012_11
Component: Carpet | Version: development version
Keywords: |
--------------------+-------------------------------------------------------
CarpetLib::interpolate_from_buffer_zones should by default set to "no"
after it is properly tested and we should think about whether we have a
good reason to not remove that paramter then again. Adding the release tag
because we don't want to have a change like this affecting two releases.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1099>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1078: ignore non-evolved points in check_GRHydro_C2P_failed
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: GRHydro |
-----------------------------------+----------------------------------------
The attached patch interfaces GRHydro with Carpet/CarpetEvolutionMask to
not abort a run if a con2prim failure happens at points that are not
active in the sense that they should not be eg. visualized.
CarpetEvolutionMask computes these points by taking the restricted region
and modifying it to take buffer zones and ghost zones for prolongation
into account.
The proposed patch uses Cray pointers (see
https://trac.einsteintoolkit.org/ticket/1065) since the Cactus flesh
routine to CCTK_VarDataPtr (for Fortran) uses them as well. This requires
extra switches for compilers to compile. For gcc the switch is "-fcray-
pointer" for intel adding "-safe_cray_ptr" might help efficiency.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1078>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#940: McLachlan dissipation error
-----------------------------------+----------------------------------------
Reporter: eschnett | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: |
-----------------------------------+----------------------------------------
A kind person tells me:
I wanted to mention what appears to be a mistake in your dissipation
calculation. If you refer to Kreiss-Oliger, or equivalently to
Dissipation/src/apply_dissipation.F77, you find that there is a sign flip
at certain orders. For example, 3rd order dissipation to be used with 2nd
order differencing, and 7th order dissipation to be used with 6th order
differencing, both need a negative coefficient. But you don't appear to
have such a minus sign (unless I missed it), and your parameter epsdiss is
constrained to be positive. It looks to me like PDdissipationNth needs to
be multiplied by something like (-1)^(fdOrder/2).
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/940>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1105: TmunuBase::support_old_CalcTmunu_mechanism to "no" and remove after release
-----------------------------------+----------------------------------------
Reporter: knarf | Owner: knarf
Type: enhancement | Status: new
Priority: minor | Milestone: ET_2012_11
Component: EinsteinToolkit thorn | Version: development version
Keywords: |
-----------------------------------+----------------------------------------
Change default of TmunuBase::support_old_CalcTmunu_mechanism to "no", mark
it depreciated for the fall 2012 ET release and remove it after that.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1105>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#815: Outlfow thorn in incoming
-------------------------+--------------------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: optional | Milestone:
Component: Other | Version:
Keywords: |
-------------------------+--------------------------------------------------
Hello all,
as discussed in the last phone call, the GT Outflow thorn has been put
into incoming (http://svn.einsteintoolkit.org/incoming/Outflow).
From the docs:
{{{
Outflow calculates the flow of rest mass density across a
SphericalSurface, eg. and apparent horizon or a sphere at
``infinity''.
}}}
It has documentation, test-cases and publications using it
(http://arxiv.org/abs/1201.4389).
Ok to include in EinsteinAnalysis?
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/815>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1061: Certificate failure
-----------------------------------+----------------------------------------
Reporter: eschnett | Owner:
Type: defect | Status: new
Priority: major | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: |
-----------------------------------+----------------------------------------
I want to access the ET svn repositories via SourceTree, a nice GUI for
managing repositories. This fails because:
Error validating server certificate for
'https://svn.einsteintoolkit.org:443':
- The certificate is not issued by a trusted authority. Use the
fingerprint to validate the certificate manually!
Certificate information:
- Hostname: svn.einsteintoolkit.org
- Valid: from Thu, 05 Jan 2012 22:31:55 GMT until Fri, 04 Jan 2013
22:31:55 GMT
- Issuer: lsu, edu
- Fingerprint:
de:b0:88:20:f7:6f:73:df:ed:ad:c2:af:7e:b7:e1:45:13:82:8a:7d
(R)eject, accept (t)emporarily or accept (p)ermanently? RA layer request
failed: OPTIONS of 'https://svn.einsteintoolkit.org/manifest': Server
certificate verification failed: issuer is not trusted
(https://svn.einsteintoolkit.org) at
/Applications/SourceTree.app/Contents/Resources/git_local/libexec/git-core
/git-svn line 2327
Unfortunately, as this is a GUI, I cannot press the "R" key here. To
simplify things for me, could you use a trusted certificate instead?
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1061>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1074: EinsteinExact code regeneration doesn't handle missing file
doc/spacetimes.tex
-----------------------------------+----------------------------------------
Reporter: eschnett | Owner:
Type: defect | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: |
-----------------------------------+----------------------------------------
While regenerating code for EinsteinExact, I received the error message
{{{
DeleteFile::nffil: File not found during
DeleteFile[../doc/spacetimes.tex].
}}}
which aborted the code generation. This error is caused when this file is
not present, and this file is then also not generated.
DeleteFile should not report an error when the file is not present.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1074>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
#1003: Add optional support for CarpetEvolutionMask to GRHydro
-----------------------------------+----------------------------------------
Reporter: rhaas | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone:
Component: EinsteinToolkit thorn | Version:
Keywords: GRHydro |
-----------------------------------+----------------------------------------
the attached patch adds access to CarpetEvolutionMask's evolution mask to
GRHydro. CarpetEvolutionMaks tracks which points on each level contribute
to the final answer, roughly speaking anything that would show up in a
call to the interpolators.
The functional change to GRHydro is trivial (just a parameter, a GF access
and a cycle in the routine that checks for C2P failures). However getting
the equivalent of CCTK_VarDataPtr to work in Fortran turned out to be
ugly. Better solutions are very welcome. Desired features are:
* easy to use
* not error prone
* allows to run GRHydro without CarpetEvolutionMask (this means inheriting
won't work)
* low memory footprint
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1003>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit