-
Notifications
You must be signed in to change notification settings - Fork 80
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
Temperature dependent single phase thermal conductivity #3135
base: develop
Are you sure you want to change the base?
Temperature dependent single phase thermal conductivity #3135
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you Sy Tuan. Lots of change in this PR, but everything looks in order.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3135 +/- ##
==========================================
Coverage ? 49.33%
==========================================
Files ? 1037
Lines ? 88227
Branches ? 0
==========================================
Hits ? 43527
Misses ? 44700
Partials ? 0 ☔ View full report in Codecov by Sentry. |
This PR is part of the project for non-linear thermal constitutive behavior from TotalEnergies. It tackles the problem of linear temperature dependent single phase effective thermal conductivity of porous medium.
thermalConductivity = defaultThermalConductivity + dThermalConductivity_dT * (T - T_reference)
where defaultThermalConductivity, dThermalConductivity_dT and T_reference are given in the xml input. The existing linear model in GEOS corresponds to the particular case of a constant thermal conductivity, i.e. dThermalConductivity_dT = 0.