-
Notifications
You must be signed in to change notification settings - Fork 88
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
Adds (S)SOR Preconditioner #1633
Conversation
499f37f
to
f5b8d2a
Compare
97d4d54
to
f8cbc05
Compare
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.
LGTM in general.
I am thinking of how to add the test for GaussSeidel in a cheap way or just do not test it, which calls SOR with relaxation 1
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.
LGTM. The config test from registry is not yet there
282ded2
to
64b4d55
Compare
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.
LGTM, only a few small comments / questions.
Note: I did not look at the correctness of the config parsing.
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.
LGTM, only a few small comments / questions.
Note: I have not looked at the correctness of the config parsing.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1633 +/- ##
===========================================
- Coverage 92.05% 90.33% -1.73%
===========================================
Files 764 776 +12
Lines 62678 63228 +550
===========================================
- Hits 57701 57116 -585
- Misses 4977 6112 +1135 ☔ View full report in Codecov by Sentry. |
only reference kernels are available
1cb476a
to
363862f
Compare
- documentation - tests - don't build upper solver if not symmetric Co-authored-by: Yu-Hsiang M. Tsai <[email protected]>
Quality Gate passedIssues Measures |
This PR adds the core and reference implementation of a (S)SOR preconditioner. It uses triangular solves to apply the preconditioner. The preconditioner is either a single lower triangular solver, or a lower triangular solve followed by an upper triangular one. Thus the preconditioner is represented as a
gko::Combination
. TheSor
class itself is only a linop factory, similar to theLu
factorization. It can be parameterized by therelaxation_factor
(0 < w < 2
), symmetric or not (sym version has the two triangular solves), and the solver factories for the upper and lower triangular matrices.This also adds a Gauss-Seidel preconditioner as a special case of SOR.
Todo: