-
Notifications
You must be signed in to change notification settings - Fork 239
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
[RM] Alternative mass balance #2914
Conversation
<type>Dirichlet</type> | ||
<parameter>pressure_bc</parameter> | ||
</boundary_condition> | ||
<!--boundary_condition> werden automatisch neumann 0 RB |
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.
Drop the commented block. Comments in English please ;)
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.
Looks good!
Add a short entry somewhere in the changelog.
fa14711
to
46ecb25
Compare
<value>0.3</value> | ||
</parameter> | ||
<!-- Model parameters --> | ||
<!--parameter> |
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.
Can this commented parameter be removed? If not, please drop a short message here why it is necessary to keep it.
<type>Constant</type> | ||
<value>0.5</value> | ||
</property> | ||
<!--property> |
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.
Can this commented property be removed? If not, please drop a short message here why it is necessary to keep it.
@@ -0,0 +1,25 @@ | |||
<?xml version="1.0"?> |
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.
Is this pvd-file used somewhere? Can it be deleted?
@@ -0,0 +1,16 @@ | |||
<?xml version="1.0"?> |
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.
Is this pvd-file used somewhere? Can it be deleted?
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 didn't check the formulas.
- Please fix the small issues with the comments in the project file.
- Please delete the pvd files if they are not needed.
Otherwise, the PR looks good. ⏩
…iles, added alternative_mass_balance_anzInterval_10 block to Tests.cmake
…masslumping, Richards_2d_geometry_OBSERVATION_POINT and deformation_dependent_porosity_swelling tests
46ecb25
to
6885b2e
Compare
OpenGeoSys development has been moved to GitLab. |
Fixes #