sst-core icon indicating copy to clipboard operation
sst-core copied to clipboard

checkpoint-wall-clock not recognizing simple seconds-only format

Open kpgriesser opened this issue 1 year ago • 0 comments

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.
- 

kpgriesser avatar Oct 14 '24 19:10 kpgriesser