-
-
Notifications
You must be signed in to change notification settings - Fork 228
Add matrix-free solver python demo #3028
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
Conversation
* add poisson matrix free demo for python * add matrix-free CG variants using KSP * add documentation to matrix-free Poisson demo * Fix typos in C++ demo * Improve documentation of Python demo * Fix MPI parallelization, remove variant using NumPy arrays * Documentation, code cleanup * Improve documentation * Adapt according to #2579 (and trigger GitHub actions again) * Call only once for the action * Replace some ghostUpdate calls with scatter_forward --------- Co-authored-by: Igor Baratta <igorbaratta@gmail.com> Co-authored-by: Garth N. Wells <gnw20@cam.ac.uk>
…nx into igor/matrix-free-demo
…nx into igor/matrix-free-demo
…o igor/matrix-free-demo
|
It turned out there were quite a lot of fixes needed to the documentation for this... but now I think is pretty good. |
|
Just a couple of typos and things that I've spotted. Will push soon. |
|
|
||
| # This matrix-free solver is now used to compute the finite element solution. | ||
| # The finite element solution's approximation error as compared with the | ||
| # exact solution is measured in the $L_2$-norm. |
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.
The solution selected here is actually exactly represented in the FE space chosen. What we're measuring is the iterative solver error as propagated to the FE solution in the L2 norm. If we chose u_D to be something not in
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 point, I think we should use something not in
Based on PR #2517 (by @juliusgh).
Updated PR adding tensor product based elements and mesh.