Skip to content

Allow the simulator to force an output write to be the final one.#5038

Open
blattms wants to merge 1 commit intoOPM:masterfrom
blattms:feature/allow-write-rsm-for-actionx-exit
Open

Allow the simulator to force an output write to be the final one.#5038
blattms wants to merge 1 commit intoOPM:masterfrom
blattms:feature/allow-write-rsm-for-actionx-exit

Conversation

@blattms
Copy link
Member

@blattms blattms commented Mar 3, 2026

The final write is special as it might create an RSM file. Up to now we created this file only for the last write of the last report step of a simulation.

This approach neglected that a simulation might finish before the last step if it encounters an EXIT keyword inside of an executed ACTIONX statement.

With this change we will also write an RSM file in this case. Such a file was missing before.

@blattms blattms added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Mar 3, 2026
@blattms
Copy link
Member Author

blattms commented Mar 3, 2026

jenkins build this opm-simulators=6894 please

@blattms blattms force-pushed the feature/allow-write-rsm-for-actionx-exit branch from dfd6978 to 9b3085d Compare March 3, 2026 16:17
The final write is special as it might create an RSM file.
Up to now we created this file only for the last write of the last
report step of a simulation.

This approach neglected that a simulation might finish before the last
step if it encounters an EXIT keyword inside of an executed ACTIONX
statement.

With this change we will also write an RSM file in this case. Such a
file was missing before.
@blattms blattms force-pushed the feature/allow-write-rsm-for-actionx-exit branch from 9b3085d to ec37934 Compare March 13, 2026 11:23
@blattms
Copy link
Member Author

blattms commented Mar 13, 2026

jenkins build this opm-simulators=6894 please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant