sst-core
sst-core copied to clipboard
checkpoint-wall-clock not recognizing simple seconds-only format
New Issue for sst-core NOTE: If you are reporting an issue for a specific element library, please submit request in ssts-element repository.
1 - Detailed description of problem or enhancement
Referencing the checkpoint-wall-period documentation:
The wall time option accepts a time formatted as HH:MM:SS, MM:SS, or SS. Alternately an integer with a time unit (h, m, or s) can be given. The simulation time option accepts an integer with a time unit of s or Hz and SI prefixes are allowed.
This works for values < 62s (and strange number) but returns a parsing error for other values.
sst --checkpoint-wall-period=62s
ERROR: Argument passed to "--checkpoint-wall-period" could not be parsed. Argument = [62s]
Valid formats are:
%H:%M:%S
%M:%S
%S
%Hh
%Mm
%Ss
Perhaps documentation can be updated for this release to read just:
The wall time option must be specified in the time format as HH:MM:SS, MM:SS, or SS.
2 - Describe how to reproduce the issue
sst --checkpoint-wall-period=62s
3 - What Operating system(s) and versions
ubuntu 22.04.5
4 - What versions of external libraries (MPI, etc.)
openmpi 4.1.2
5 - Provide sha1 of all relevant SST repositories (sst-core, sst-elements, etc)
commit 335de84417671898414e7f4d242c5ab109d5f963 (HEAD -> devel, origin/devel)
Author: Scott Hemmert <[email protected]>
Date: Mon Oct 7 16:30:05 2024 -0600
Bug fixes for parallel checkpointing. (#1158)
- Fixes #1156 - Initial Sync interval needed to take Checkpoint time into account.
- Fixes #1157 - Moved barrier for Exit restart to the proper place.
-