Skip to content

Tag Potentially Unused Parameter as Maybe_Unused#6984

Merged
bska merged 1 commit intoOPM:masterfrom
bska:maybe-unused-dt
Apr 15, 2026
Merged

Tag Potentially Unused Parameter as Maybe_Unused#6984
bska merged 1 commit intoOPM:masterfrom
bska:maybe-unused-dt

Conversation

@bska
Copy link
Copy Markdown
Member

@bska bska commented Apr 13, 2026

The time step size parameter is referenced only when reservoir coupling is enabled, which is not the case for sequential builds.

@bska bska added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Apr 13, 2026
@bska
Copy link
Copy Markdown
Member Author

bska commented Apr 13, 2026

jenkins build this serial please

@bska bska force-pushed the maybe-unused-dt branch from 877d3fe to e6f633b Compare April 14, 2026 09:01
@bska
Copy link
Copy Markdown
Member Author

bska commented Apr 14, 2026

jenkins build this serial please

The time step size parameter is referenced only when reservoir
coupling is enabled, which is not the case for sequential builds.
@bska bska force-pushed the maybe-unused-dt branch from e6f633b to a089e60 Compare April 14, 2026 12:40
@bska
Copy link
Copy Markdown
Member Author

bska commented Apr 14, 2026

jenkins build this serial please

@akva2 akva2 added this to the Release 2026.04 milestone Apr 15, 2026
@bska
Copy link
Copy Markdown
Member Author

bska commented Apr 15, 2026

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

@bska bska merged commit 7ba73e1 into OPM:master Apr 15, 2026
2 checks passed
@bska bska deleted the maybe-unused-dt branch April 15, 2026 07:48
michal-toth pushed a commit to michal-toth/opm-simulators that referenced this pull request Apr 17, 2026
Tag Potentially Unused Parameter as Maybe_Unused
@michal-toth michal-toth added the backported Release managers use this at their discretion label Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported Release managers use this at their discretion 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.

3 participants