#910: Cactus does not detect circular schedule items --------------------+------------------------------------------------------- Reporter: rhaas | Owner: Type: defect | Status: new Priority: minor | Milestone: Component: Cactus | Version: Keywords: | --------------------+------------------------------------------------------- Right now none of the three schedule items below (which should all be flagged as faulty I think), triggers any warning during compile or run time: {{{ schedule group ordertest at initial { } "test schedule sorter"
schedule ADMBase_CartesianMinkowski in ordertest after ADMBase_LapseOne { LANG: C } "Set the metric and extrinsic curvature to cartesian minkowski values"
schedule ADMBase_LapseOne in ordertest after ADMBase_CartesianMinkowski { LANG: C } "Set the lapse to 1 at all points"
SCHEDULE ADMBase_SetShiftStateOn in ordertest after ADMBase_SetShiftStateOn { LANG: C } "Set the shift_state variable to 1" }}}
I attach: * a patch against ADMBase/schedule.ccl to demonstrate the output * stdout when one runs Exact/Schwarzschild_EF with the faulty schedule.ccl * a patch against src/schedule/ScheduleSorter.c and Creator that adds error checks for this situation (the part inside of TEST_SCHEDULECREATOR is for the stand-alone test inside to compile)
#910: Cactus does not detect circular schedule items ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: review Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: ---------------------+------------------------------------------------------ Changes (by rhaas):
* status: new => review
#910: Cactus does not detect circular schedule items ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: review Priority: minor | Milestone: ET_2012_11 Component: Cactus | Version: Resolution: | Keywords: ---------------------+------------------------------------------------------ Changes (by knarf):
* milestone: => ET_2012_11
#910: Cactus does not detect circular schedule items ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: reviewed_ok Priority: minor | Milestone: ET_2012_11 Component: Cactus | Version: Resolution: | Keywords: ---------------------+------------------------------------------------------ Changes (by eschnett):
* status: review => reviewed_ok
Comment:
The first patch is okay to apply.
The second patch does not say to which thorn it should be applied. The patch is okay if it is applied to thorn in CactusTest (but not to the flesh or e.g. an Einstein Toolkit thorn).
#910: Cactus does not detect circular schedule items ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: reviewed_ok Priority: minor | Milestone: ET_2012_11 Component: Cactus | Version: Resolution: | Keywords: ---------------------+------------------------------------------------------
Comment (by rhaas):
ordertest_schedule.ccl.patch is not intended to be applied. It was only provided so that others could test the reported problem. It is a patch against ADMBase.
I applied the first patch detect_circular_dependencies.patch as rev 4832 of the flesh.
#910: Cactus does not detect circular schedule items ---------------------+------------------------------------------------------ Reporter: rhaas | Owner: Type: defect | Status: closed Priority: minor | Milestone: ET_2012_11 Component: Cactus | Version: Resolution: fixed | Keywords: ---------------------+------------------------------------------------------ Changes (by rhaas):
* status: reviewed_ok => closed * resolution: => fixed
trac@lists.einsteintoolkit.org
-
Einstein Toolkit