ert icon indicating copy to clipboard operation
ert copied to clipboard

Warning if the simulation folder is not empty

Open DanSava opened this issue 1 year ago • 2 comments

Issue Users should receive a warning that the simulation folder used for the optimization is not empty. And should be given a choice to delete it via a dialog or checkbox in the GUI. In the CLI a warning should be given and delete the folder before the run starts.

DanSava avatar Sep 18 '24 06:09 DanSava

A warning here is probably a good idea. We should make sure the user know that deletion is slow, and should probably make them do it on their own, but we can have a prompt whether they would like to continue.

oyvindeide avatar Oct 02 '24 11:10 oyvindeide

Also be aware that ert already has functionality for this, so if we can align the two, that is beneficial.

oyvindeide avatar Oct 05 '24 17:10 oyvindeide

For this we could override the check_if_runpath_exists() in the EverestRunModel, then in the everest execution flow, create an everest baserunmodel right after the everest config is initialized, and invoke check_if_runpath_exists(). I'm assuming there is no existing logic for checking if the runpath exists in Everest.

If you can get to know the range of GEO_ID and REALIZATION, you can do a more thorough check, but I think it would also be fine to start with something simple, like just checking if the simulation output directory exists.

yngve-sk avatar Nov 05 '24 10:11 yngve-sk

Open question/edge case this: Will a terminal prompt mess up someones workflow? If they have a bash script they expect to just run, and everest run is part of that, the prompt would halt it forever

yngve-sk avatar Nov 12 '24 13:11 yngve-sk