DurhamARC / classroom-abm

Agent-based modelling for a classroom
MIT License
2 stars 2 forks source link

Adapt automation to consider the results from all previous iterations… #149

Closed parnumeric closed 2 years ago

parnumeric commented 2 years ago

… instead of only the last run

parnumeric commented 2 years ago

Closes #135

parnumeric commented 2 years ago

The $MERGE_FILE to store all previous concatenated merged_mses.csv must not be touched in advance, because the program will fail reading the empty file. In fact, it should be removed if exists. So, the $MERGE_FILE must not exist before the simulation.

alisonrclarke commented 2 years ago

The $MERGE_FILE to store all previous concatenated merged_mses.csv must not be touched in advance, because the program will fail reading the empty file. In fact, it should be removed if exists. So, the $MERGE_FILE must not exist before the simulation.

Can you add this detail to the Hamilton README file please? Or could the script delete it automatically, or fail if it exists?

alisonrclarke commented 2 years ago

Build failure was because secrets aren't available in @parnumeric's copy of the repo. #150 is a replacement branched from the DurhamARC org, so closing this PR.