-
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
[PL/HT] Fix computation of darcy velocity. #2124
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.
Needs a ctest catching this...
Can be an existing test, but check for velocity explicitly.
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 have another PR for the staggered scheme too.
Since the HT process is not tested for some time and the current results differ from the expected values I'll add more ctest to ensure the process will compute correct results. @wenqing Is your PR for the staggered scheme handling the Darcy velocity? Then maybe we can merge our works? |
@TomFischer Yes, it is. My PR is for the staggered scheme handling the Darcy velocity. Therefore, my PR should not have the same changes as that in this PR. Should I re-sent this PR to your branch for this PR? |
@wenqing I think we can let them as separate PRs. |
Corrected access to the local pressure and temperature variables.
be93c68
to
c9e1b43
Compare
c9e1b43
to
953a02a
Compare
Also relax the tolerance for temperature in the test.
OpenGeoSys development has been moved to GitLab. |
Corrected access to the local pressure and temperature variables.
Fixes issue #2054.