#317: Unproper bevaviour of sim submit when changing the parfile ----------------------------------------------+----------------------------- Reporter: alexander.beck-ratzka@… | Owner: mthomas Type: defect | Status: new Priority: major | Milestone: Component: SimFactory | Version: ET_2010_11 Keywords: | ----------------------------------------------+----------------------------- If I make changes to the parfile of my simulation, and call then simfactory submit with the option --parfile=newpar, simfactory does not take this parfile. Instead of this, simfactory uses an old parfile from the old output-xxxx directory in my simualations direrctory.
This usage of the old parfile might be wished. However, if this is the case, simfactory must complain if invoked with the option "parfile=...". But simfactory does not complain.
In my case, the parfile has been modfied, but I am using the same name.
#317: Unproper bevaviour of sim submit when changing the parfile -----------------------------------------------+---------------------------- Reporter: alexander.beck-ratzka@… | Owner: mthomas Type: defect | Status: new Priority: major | Milestone: Component: SimFactory | Version: ET_2010_11 Resolution: | Keywords: -----------------------------------------------+----------------------------
Comment (by hinder):
I can reproduce this problem:
cp simfactory/etc/parfiles/basicrecover.par recover1.par cp simfactory/etc/parfiles/basicrecover.par recover2.par sim create-submit testpar --parfile recover1.par --procs 1 --walltime 1:00:00 # Wait for simulation to finish sim submit testpar --parfile recover2.par # Wait for simulation to finish ls -1 ~/Simulations/testpar/output-????/*.par
Gives:
/Users/ian/Simulations/testpar/output-0000/recover1.par /Users/ian/Simulations/testpar/output-0001/recover1.par
#317: Unproper bevaviour of sim submit when changing the parfile -----------------------------------------------+---------------------------- Reporter: alexander.beck-ratzka@… | Owner: mthomas Type: defect | Status: closed Priority: major | Milestone: Component: SimFactory | Version: ET_2010_11 Resolution: duplicate | Keywords: -----------------------------------------------+---------------------------- Changes (by hinder):
* status: new => closed * resolution: => duplicate
Comment:
I tried to fix this, but I found that the design of simfactory is that the parfile is not changed between restarts. Specifically, much of the logic for performing substitutions in parfiles is in the create command, not the submit command, and it does not seem to be trivial to change this. Given this, I would say that the main bug is that simfactory doesn't abort when you give it options it doesn't understand. This is essentially the same as
#221 - Simfactory should complain about unused arguments
which I have just modified to include a comment about this applying to options as well.
Since you cannot change the parfile between restarts, I have created a new enhancement request
#487 - SimFactory should allow steering of parameters between restarts
This will require a bit of internal redesign. I am closing this ticket, since it is now covered by the other two.
trac@lists.einsteintoolkit.org
-
Einstein Toolkit