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 GMRES restart option #2910
add GMRES restart option #2910
Conversation
add gmres restart test1 set gmres restart test2 add restart option for all Preconditioners (final)
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.
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.
Clang-format is required on the whole change.
Documention.
Testcase?
Looks ok otherwise. Nice addition!
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.
When Dimas issues are resolved and tests are green: ⏩
} | ||
|
||
template <> | ||
void EigenIterativeLinearSolver<Eigen::GMRES<EigenMatrix::RawMatrixType, Eigen::IncompleteLUT<double>>>::setRestart(int const restart) |
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.
@joergbuchwald clang-format. Use pre-commit checks.
@joergbuchwald Small testcase using the option, something from SteadyStateDiffusion, for example, would be good. |
OpenGeoSys development has been moved to GitLab. |
add restart option for the GMRES solver (Eigen-unsupported).
It sets the number of iterations before a restart is conducted. It can improve the convergence behavior. However, increasing the value comes with a tradeoff in time and memory. First tests suggest that the default value (30) is already quite optimal in most cases.