Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
kpgriesser opened this issue Oct 14, 2024 · 0 comments
Open

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

kpgriesser opened this issue Oct 14, 2024 · 0 comments

Comments

@kpgriesser
Copy link

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.
- 
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant