Skip to content

Remove unused GMRES option in curl-curl MLMG preconditioner - #7216

Merged
JustinRayAngus merged 1 commit into
BLAST-WarpX:developmentfrom
RemiLehe:remove_gmres_in_curlcurl_pc
Aug 28, 2026
Merged

Remove unused GMRES option in curl-curl MLMG preconditioner#7216
JustinRayAngus merged 1 commit into
BLAST-WarpX:developmentfrom
RemiLehe:remove_gmres_in_curlcurl_pc

Conversation

@RemiLehe

Copy link
Copy Markdown
Member

The curl-curl preconditioner had an undocumented option to use GMRES (optionally preconditioned by MLMG) instead of MLMG directly, i.e. GMRES as a preconditioner inside the outer GMRES solver. This was never used in practice and was not documented, so remove it along with the parsing of pc_curl_curl_mlmg.use_gmres and pc_curl_curl_mlmg.use_gmres_pc.

The curl-curl preconditioner had an undocumented option to use GMRES
(optionally preconditioned by MLMG) instead of MLMG directly, i.e. GMRES
as a preconditioner inside the outer GMRES solver. This was never used
in practice and was not documented, so remove it along with the parsing
of pc_curl_curl_mlmg.use_gmres and pc_curl_curl_mlmg.use_gmres_pc.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@RemiLehe
RemiLehe requested a review from debog August 28, 2026 14:15
@JustinRayAngus
JustinRayAngus merged commit db96090 into BLAST-WarpX:development Aug 28, 2026
51 checks passed
@ax3l ax3l added component: implicit solvers Anything related to implicit solvers cleaning Clean code, improve readability labels Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleaning Clean code, improve readability component: implicit solvers Anything related to implicit solvers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants