-
Notifications
You must be signed in to change notification settings - Fork 242
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
Create a Cookbook for the Tian Parameterization Reactive Fluid Transport Model #5746
Create a Cookbook for the Tian Parameterization Reactive Fluid Transport Model #5746
Conversation
3fe7b6f
to
e14f4a8
Compare
7d53a03
to
c952f26
Compare
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.
@danieldouglas92 - Thanks for this contribution, really exciting to see the results of all these pieces coming together. An initial round of comments on the PRM file is below.
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
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.
@danieldouglas92 - A few more comments to address, more to come later on. Overall the structure and content is in pretty good shape. The figure looks great, exciting results!
One of the main things that I think should be updated is a much more detailed description of the choices for both physical and numerical, given that these models are highly nonlinear and small variations in some parameters can lead to significant variations in model behavior. In general, I think it will be easiest to do this in the PRM file and keep the cookbook text more focused on what is being observed.
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/schematic-diagram.png
Outdated
Show resolved
Hide resolved
.../tian_parameterization_kinematic_slab/single_phase_tian_parameterization_kinenmatic_slab.prm
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
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.
A few more comments. I focused on the text in the manual rather than on the input file.
cookbooks/tian_parameterization_kinematic_slab/doc/schematic_diagram.png
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...parameterization_kinematic_slab/uncoupled-two-phase-tian-parameterization-kinematic-slab.prm
Show resolved
Hide resolved
e4fd9d4
to
c5c9176
Compare
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.
@danieldouglas92 - Thanks for all of the changes so far, and a few more comments below. I think after this the cookbook should be ready or nearly ready from my side.
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Outdated
Show resolved
Hide resolved
cookbooks/tian_parameterization_kinematic_slab/doc/tian_parameterization_kinematic_slab.md
Show resolved
Hide resolved
...parameterization_kinematic_slab/uncoupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
71e33ec
to
d35377a
Compare
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 mostly have a few comments on your test files, other wise this looks good!
tests/uncoupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
tests/uncoupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
tests/coupled_two_phase_tian_parameterization_kinematic_slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
d35377a
to
94c6746
Compare
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.
@danieldouglas92 - Just a few more minor comments, and then this is ready to merge from my side!
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Outdated
Show resolved
Hide resolved
...n_parameterization_kinematic_slab/coupled-two-phase-tian-parameterization-kinematic-slab.prm
Show resolved
Hide resolved
94c6746
to
4d9f83a
Compare
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.
👍 Good to go from my side once the testers pass.
/rebuild |
4d9f83a
to
3d91168
Compare
3d91168
to
58a1b62
Compare
Create a cookbook of simple subduction that demonstrates how the Tian approximation is used to simulate hydration/dehydration reactions between a solid and fluid phase. This cookbook investigates the upper 20 km of a slab that is comprised of 3 hydrated lithologies: sediment, MORB, and gabbro that subducts through a hot peridotite mantle. The slab enters the model from the left through fixed composition and temperature boundary conditions. It moves at 5 cm/yr due to kinematic boundary conditions on all sides, and heats up with time causing the dehydration of the lithologies within the slab. As a result, the overlying peridotite mantle is partially hydrated. A slab dip of 45 degrees is achieved by orienting the gravity vector at an angle of 45 degrees from vertical (see schematic diagram).
One input file models an uncoupled system, where the solid phase is unaffected by the fluid phase. The second input file models a coupled system, where the solid and fluid phases directly impact each other.