llnl / llnl/smith

Add ability to restart based on different directory than outputting to

Open
#730 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

CS output quick
Dominant language
C++
Stars
245
Forks
36
Avg merge
8d 23h
Merged PRs (30d)
4

Description

We are currently copying the previous run's output directory into the new directory for testing purposes. This may cause problems in the future when data outputs get large.

Do we want to support the following command line option:

```
serac -i default.lua --input-dir=original_output_dir --output=new_output_dir
```

Where `input-dir` (probably come up with a better name) is only used for restart capabilities.

https://github.com/LLNL/serac_tests/pull/11/files#r907880313

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the serac command-line handling for --output and the existing restart path; the issue names no files or tests. Determine how a separate input directory should be selected for restart data while new results use the output directory, then verify that restarting works without copying the prior output.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
cli, hpc
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.