-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add Neumann BC in the LinearFV system #30146 #30147
base: next
Are you sure you want to change the base?
Conversation
Job Documentation, step Docs: sync website on 438a673 wanted to post the following: View the site here This comment will be updated on new commits. |
Job Coverage, step Generate coverage on 438a673 wanted to post the following: Framework coverage
Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
… or a gradient extrapolation. Fixes issues with LinearFVDiffusion::computeBoundaryRHSContribution(), where for non-orthogonal meshes with Neumann BC an extra contribution was added that resulted in wrong results.
…e the boundary value from the flux
…der for orthogonal meshes and first order for non-orthogonal meshes
6728f2f
to
7801fa2
Compare
…nality. Obtain second order convergence for 2d diffusion in non-orthogonal meshes
1c9246c
to
d3d3dea
Compare
Job Precheck, step Clang format on 316740c wanted to post the following: Your code requires style changes. A patch was auto generated and copied here
Alternatively, with your repository up to date and in the top level of your repository:
|
Closes #30146