Skip to content
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

Hydrothermal conductivity with depth function with fastscape #14

Open
wants to merge 47 commits into
base: main
Choose a base branch
from

Conversation

RichardBoelz
Copy link

Hello Anne, this is my most recent branch, where the fastscape version and the depth with respect to the current surface is included.

I was not sure what to do in case the parameter 'Use depth of sea for hydrothermal cooling approximation' is set to true, but there is no mesh deformation. Because then nothing would change during the computation of the new thermal conductivity (because depth_of_sea = 0 && use_depth_of_sea = true). I just threw an error in this case.

I really tried to have a nice structure of the code, but we already have several cases to check, therefore, it wasn't too easy.

Before your first pull request:

For all pull requests:

For new features/models or changes of existing features:

  • I have tested my new feature locally to ensure it is correct.
  • I have created a testcase for the new feature/benchmark in the tests/ directory.
  • I have added a changelog entry in the doc/modules/changes directory that will inform other users of my change.

bbxriboe and others added 30 commits September 19, 2023 17:04
bbxriboe and others added 17 commits September 28, 2023 13:59
Remove comments and additional lines

Add asserts
Second batch of squashed fastscape commits.

Refactor saving/loading files for restart

add paramters for erosional base level differing from sea level

third batch of squashed fastscape commits.

Expand on plugin description

Fourth batch of squashed fastscape commits.

Update include/aspect/mesh_deformation/fastscape.h

Co-authored-by: Wolfgang Bangerth <[email protected]>

Update comment

Update reptitions types and add python script to rewrite fastscape visualizations with time.

Add get_end_time function

Update cookbook and add check in viscoplastic

Update sediment rain times

Update cmakelists

Remove commented line

Fix library link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant