[ET Trac] [Einstein Toolkit] #616: Carpet breaks backwards-compatibility with parameter files
Einstein Toolkit
trac-noreply at einsteintoolkit.org
Mon Oct 17 09:16:22 CDT 2011
#616: Carpet breaks backwards-compatibility with parameter files
---------------------+------------------------------------------------------
Reporter: hinder | Owner: eschnett
Type: defect | Status: new
Priority: major | Milestone: ET_2011_10
Component: Carpet | Version:
Resolution: | Keywords:
---------------------+------------------------------------------------------
Comment (by barry.wardell):
Replying to [comment:3 eschnett]:
> I don't think that's worthwhile. This parameter doesn't influence
correctness of the evolution, so let's keep things simple. 0 is already an
unlikely value, most people would use 253 or so.
OK.
> If you want to propose a patch, then it should probably run in
PARAMCHECK and use CCTK_ParameterQueryTimesSet. Please test
checkpointing/recovery as well to ensure the warning works as intended.
>
> Setting parameters which have no effect is usually not diagnosed in
Cactus; this should therefore likely be a low-priority warning (maybe
level 2).
Now that I think about it more, this seems like feature which would be
better suited as a part of Cactus. It happens quite often that parameters
are deprecated, so having a standardised way of denoting this in the
param.ccl would be better than implementing the code manually each time.
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/616#comment:4>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
More information about the Trac
mailing list