Skip to content
Snippets Groups Projects

Only dump the used/unused parameter yaml files when *not* restarting a run

Merged Matthieu Schaller requested to merge param_dumps into master

This is a trivial change but thought you may have a better suggestion.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • I am actually surprised we haven't seen issues with this yet since all ranks were attempting to write.

  • Agreed... I've checked these files from time to time and never seen any garbled output. Must be fast to write and close and each process went past at slightly different times, so no simultaneous opens (alternatively I wonder what the IO buffering does when you open a file, and only flush to it on close, the contents is less than 4096 characters, so probably looks like a single write).

    No better ideas, so accepting.

  • Peter W. Draper mentioned in commit 2b948447

    mentioned in commit 2b948447

Please register or sign in to reply
Loading