-
Notifications
You must be signed in to change notification settings - Fork 89
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
feat: Temperature dependent single phase thermal conductivity #3135
Merged
CusiniM
merged 60 commits into
develop
from
feature/sytuan/temperatureDependentThermalConductivity
Sep 5, 2024
Merged
Changes from 53 commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
901e4b7
prepare for implementing linear solid thermal conductivity
sytuannguyen 158bb3e
update the thermal conductivity constitutive class
sytuannguyen 5303002
update the name of the thermal conductivity model in xml files
sytuannguyen 89b7909
update thermal model name in srt and py files
sytuannguyen fde823e
add thermal model name to coupled constitutive model
sytuannguyen 541976e
add cond gradient attribute and update xmls
sytuannguyen 3e3676a
update thermal single phase FVM kernel w.r.t. non const thermal condu…
sytuannguyen 233f100
add dThermalCond_dT to single phase thermal flow solver
sytuannguyen 4219e49
fix floating point exception
sytuannguyen 6ae57b7
add thermal conductivity derivative to kernel
sytuannguyen 4de8fab
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen 826172b
fix conflict
sytuannguyen 0c341e9
uncrustify_style
sytuannguyen 5aa2b77
add doc and python script for validation
sytuannguyen 26b7ac4
update thermal unit tests
sytuannguyen a024785
add integrated test and xmls for benchmark
sytuannguyen cf70f7d
update doc and Python script
sytuannguyen 39d8e3f
fix unit test
sytuannguyen 2f9a18e
more fix for unit test
sytuannguyen fd3a389
fix set apply default vector
sytuannguyen 5d6ac8f
uncrustify_style
sytuannguyen 20cea11
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen aeace94
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen 1e39149
merge develop
sytuannguyen 8aa1c83
add ThermalCompressibleSolid class
sytuannguyen 18705ed
update xml
sytuannguyen f0caa6c
uncrustify_style
sytuannguyen ef365ac
fix an unit test
sytuannguyen 9180501
add ThermoPoroMechanics class
sytuannguyen 88abd53
uncrustify_style
sytuannguyen 9453bc8
update the validation examples
sytuannguyen 31d7853
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen 9a3996a
fix python scripts
sytuannguyen 894be09
update linear thermal diffusion example
sytuannguyen 8df82ee
update a python script
sytuannguyen 9ab1382
simplification
sytuannguyen 50d6048
update xmls
sytuannguyen 7af4efd
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen 168df8e
update xmls
sytuannguyen ed5f588
remove redundant files
sytuannguyen 8b8303f
some minor fixs
sytuannguyen bd3bb35
fix an unit test
sytuannguyen bd096dd
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
jhuang2601 c3152ab
add smoke tests for integratedTest
jhuang2601 8fadb85
add a doc for this new model
jhuang2601 5c6f798
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen f296222
address comments
sytuannguyen 1fe01a9
Merge branch 'feature/sytuan/temperatureDependentThermalConductivity'…
sytuannguyen c28eeba
address comments
sytuannguyen 5a059f7
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
sytuannguyen 4810e0f
address comments
sytuannguyen 07dfcbf
merge develop and resolve conflict
sytuannguyen bdd425b
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
jhuang2601 8b94b00
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
rrsettgast 2647b31
update integratedTests tags
jhuang2601 14b7ce3
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
jhuang2601 cc215e1
update COPYRIGHT statement for one cpp file
jhuang2601 53174a0
Merge branch 'develop' into feature/sytuan/temperatureDependentTherma…
jhuang2601 60e4584
update BASELINE_NOTES.md
jhuang2601 6e7b2f6
update .integrated_tests.yaml
jhuang2601 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
50 changes: 50 additions & 0 deletions
50
...glePhaseFlow/thermalCompressible_temperatureDependentVolumetricHeatCapacity_benchmark.xml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
<?xml version="1.0" ?> | ||
|
||
<Problem> | ||
<Included> | ||
<File name="./thermalCompressible_2d_base.xml" /> | ||
</Included> | ||
|
||
<ElementRegions> | ||
<CellElementRegion | ||
name="region" | ||
cellBlocks="{ cb }" | ||
materialList="{ fluid, rock_temperatureDependentVolumetricHeatCapacity, thermalCond_linear }"/> | ||
</ElementRegions> | ||
|
||
<Mesh> | ||
<InternalWellbore | ||
name="mesh1" | ||
elementTypes="{ C3D8 }" | ||
radius="{ 0.1, 0.106, 0.133, 1.0 }" | ||
theta="{ 0, 90 }" | ||
zCoords="{ 0, 0.1 }" | ||
nr="{ 3, 10, 50 }" | ||
nt="{ 80 }" | ||
nz="{ 1 }" | ||
trajectory="{ { 0.0, 0.0, 0.0 }, | ||
{ 0.0, 0.0, 0.1 } }" | ||
autoSpaceRadialElems="{ 0, 0, 1 }" | ||
cellBlockNames="{ cb }" | ||
/> | ||
</Mesh> | ||
|
||
<Events | ||
maxTime="1e5"> | ||
<PeriodicEvent | ||
name="outputs" | ||
timeFrequency="1e4" | ||
target="/Outputs/vtkOutput"/> | ||
|
||
<PeriodicEvent | ||
name="solverApplications" | ||
maxEventDt="1e3" | ||
target="/Solvers/singleFlow"/> | ||
|
||
<!--PeriodicEvent | ||
name="restarts" | ||
timeFrequency="1e5" | ||
target="/Outputs/restartOutput"/--> | ||
</Events> | ||
|
||
</Problem> |
50 changes: 50 additions & 0 deletions
50
.../singlePhaseFlow/thermalCompressible_temperatureDependentVolumetricHeatCapacity_smoke.xml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
<?xml version="1.0" ?> | ||
|
||
<Problem> | ||
<Included> | ||
<File name="./thermalCompressible_2d_base.xml" /> | ||
</Included> | ||
|
||
<ElementRegions> | ||
<CellElementRegion | ||
name="region" | ||
cellBlocks="{ cb }" | ||
materialList="{ fluid, rock_temperatureDependentVolumetricHeatCapacity, thermalCond_linear }"/> | ||
</ElementRegions> | ||
|
||
<Mesh> | ||
<InternalWellbore | ||
name="mesh1" | ||
elementTypes="{ C3D8 }" | ||
radius="{ 0.1, 0.106, 0.133, 1.0 }" | ||
theta="{ 0, 90 }" | ||
zCoords="{ 0, 0.1 }" | ||
nr="{ 3, 10, 50 }" | ||
nt="{ 8 }" | ||
nz="{ 1 }" | ||
trajectory="{ { 0.0, 0.0, 0.0 }, | ||
{ 0.0, 0.0, 0.1 } }" | ||
autoSpaceRadialElems="{ 0, 0, 1 }" | ||
cellBlockNames="{ cb }" | ||
/> | ||
</Mesh> | ||
|
||
<Events | ||
maxTime="3e3"> | ||
<PeriodicEvent | ||
name="outputs" | ||
timeFrequency="1e3" | ||
target="/Outputs/vtkOutput"/> | ||
|
||
<PeriodicEvent | ||
name="solverApplications" | ||
maxEventDt="1e3" | ||
target="/Solvers/singleFlow"/> | ||
|
||
<PeriodicEvent | ||
name="restarts" | ||
timeFrequency="1e3" | ||
target="/Outputs/restartOutput"/> | ||
</Events> | ||
|
||
</Problem> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
why? is this case taking 100 nonlinear iterations?
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.
20 would be sufficient
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.
This number is to ensure the convergence of the benchmark case
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.
I should converge in 20. if not, then something is wrong.
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.
I can confirm that it converges in 20.