Skip to content

Include Time of Day in Report Step List#4830

Merged
bska merged 1 commit into
OPM:masterfrom
bska:include-time-in-report-step-list
Nov 17, 2025
Merged

Include Time of Day in Report Step List#4830
bska merged 1 commit into
OPM:masterfrom
bska:include-time-in-report-step-list

Conversation

@bska
Copy link
Copy Markdown
Member

@bska bska commented Nov 11, 2025

When the user passes the -l option, the convertECL utility generates a list of restart steps in a unified restart file. At present, this list has the following format:

Report step number:    0   Date: 1997/11/06
Report step number:    1   Date: 1997/11/14
[...]
Report step number:  239   Date: 2006/09/14
Report step number:  240   Date: 2006/10/01
Report step number:  241   Date: 2006/10/10

This commit extends the list to also include the time of day, which is particularly useful if there are multiple restart steps in a single day. We also switch to outputting month names instead of month numbers similarly to how the rest of the OPM suite reports dates. With these changes, the above list will be printed as

Report step number:    0   Date: 06-Nov-1997 00:00:00
Report step number:    1   Date: 14-Nov-1997 00:00:00
[...]
Report step number:  239   Date: 14-Sep-2006 00:00:00
Report step number:  240   Date: 01-Oct-2006 00:00:00
Report step number:  241   Date: 10-Oct-2006 00:00:00

instead.

@bska bska added the manual:enhancement This is an enhancement/improvent that needs to be documented in the manual label Nov 11, 2025
@bska
Copy link
Copy Markdown
Member Author

bska commented Nov 11, 2025

jenkins build this please

@bska bska force-pushed the include-time-in-report-step-list branch 4 times, most recently from 4a6daf9 to 9e49e41 Compare November 13, 2025 15:11
Copy link
Copy Markdown
Member

@akva2 akva2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one (old) grammar thing.

Comment thread test_util/convertECL.cpp Outdated
When the user passes the '-l' option, the convertECL utility
generates a list of restart steps in a unified restart file.  At
present, this list has the following format:

    Report step number:    0   Date: 1997/11/06
    Report step number:    1   Date: 1997/11/14
    [...]
    Report step number:  239   Date: 2006/09/14
    Report step number:  240   Date: 2006/10/01
    Report step number:  241   Date: 2006/10/10

This commit extends the list to also include the time of day, which
is particularly useful if there are multiple restart steps in a
single day.  We also switch to outputting month names instead of
month numbers similarly to how the rest of the OPM suite reports
dates.  With these changes, the above list will be printed as

    Report step number:    0   Date: 06-Nov-1997 00:00:00
    Report step number:    1   Date: 14-Nov-1997 00:00:00
    [...]
    Report step number:  239   Date: 14-Sep-2006 00:00:00
    Report step number:  240   Date: 01-Oct-2006 00:00:00
    Report step number:  241   Date: 10-Oct-2006 00:00:00

instead.
@bska bska force-pushed the include-time-in-report-step-list branch from 9e49e41 to af9f948 Compare November 14, 2025 10:02
@bska
Copy link
Copy Markdown
Member Author

bska commented Nov 17, 2025

jenkins build this please

@bska
Copy link
Copy Markdown
Member Author

bska commented Nov 17, 2025

PR approved and build check is green. I'll merge into master.

@bska bska merged commit 9edf803 into OPM:master Nov 17, 2025
2 checks passed
@bska bska deleted the include-time-in-report-step-list branch November 17, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:enhancement This is an enhancement/improvent that needs to be documented in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants