Added a run label to the parameter file that gets written to the snapshot headers. (!817) · Merge requests · SWIFT / SWIFTsim · GitLab

Snippets Groups Projects

Merged Matthieu Schaller requested to merge run_labels into master 6 years ago

1 unresolved thread

Does what it says on the tin.

Activity

Matthieu Schaller added 1 commit 6 years ago
added 1 commit

217a7594 - Write the run label at the same in the single_io case as in the other io modules.

Compare with previous version
Josh Borrow @jborrow 6 years ago

Developer

LGTM, I'll test this and add it to swiftsimio before you merge?
Matthieu Schaller added 1 commit 6 years ago
added 1 commit

152a3422 - Added run labels to the ICs for which a well-known name exists.

Compare with previous version
Matthieu Schaller assigned to @pdraper 6 years ago

assigned to @pdraper
Matthieu Schaller @matthieu 6 years ago

Author Owner

@pdraper that should all be safe but given there is some i/o string manipulation involved I welcome an extra pair of eyes. Thanks!
Peter W. Draper @pdraper started a thread on an old version of the diff 6 years ago

src/engine.c

   /* Make the space link back to the engine. */
   s->e = e;
   /* Read the run label */
   memset(e->run_name, 0, PARSER_MAX_LINE_SIZE);
   parser_get_opt_param_string(params, "MetaData:run_name", e->run_name, "");

Matthieu Schaller added 1 commit 6 years ago

added 1 commit

7ac476f0 - Set a default non-empty run name.

Compare with previous version

Matthieu Schaller added 1 commit 6 years ago

added 1 commit

03ebd831 - Set a default non-empty run name.

Compare with previous version

Peter W. Draper @pdraper 6 years ago

Owner

Thanks, all good to go now.

Peter W. Draper merged 6 years ago

merged

Peter W. Draper mentioned in commit 6 years ago

mentioned in commit fc7c339d

Please register or sign in to reply