Skip to content

TUNINGDP for temperature#4924

Open
totto82 wants to merge 1 commit intoOPM:masterfrom
totto82:tuningdt
Open

TUNINGDP for temperature#4924
totto82 wants to merge 1 commit intoOPM:masterfrom
totto82:tuningdt

Conversation

@totto82
Copy link
Copy Markdown
Member

@totto82 totto82 commented Jan 23, 2026

No description provided.

@totto82 totto82 added the manual:enhancement This is an enhancement/improvent that needs to be documented in the manual label Jan 23, 2026
@totto82
Copy link
Copy Markdown
Member Author

totto82 commented Jan 23, 2026

jenkins build this opm-simulators=6756 please

1 similar comment
@totto82
Copy link
Copy Markdown
Member Author

totto82 commented Jan 23, 2026

jenkins build this opm-simulators=6756 please

Copy link
Copy Markdown
Contributor

@svenn-t svenn-t left a comment

Choose a reason for hiding this comment

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

Just a small comment that depends on how the default is handled in OPM/opm-simulators#6756. Otherwise it looks good to me!

"name": "TRGDDT",
"value_type": "DOUBLE",
"dimension": "Temperature",
"default": 0.1
Copy link
Copy Markdown
Contributor

@svenn-t svenn-t Jan 27, 2026

Choose a reason for hiding this comment

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

If a safeguard is put in place to not enforce this default for simulations without temperature model, then this default is OK.

@totto82 totto82 marked this pull request as ready for review January 27, 2026 12:12
@totto82
Copy link
Copy Markdown
Member Author

totto82 commented Jan 27, 2026

jenkins build this opm-simulators=6756 please

1 similar comment
@totto82
Copy link
Copy Markdown
Member Author

totto82 commented Jan 28, 2026

jenkins build this opm-simulators=6756 please

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