Skip to content

Make coloring as default if we are using petscASM.#5211

Open
Molin-Han wants to merge 5 commits into
mainfrom
molin/ASMColoring
Open

Make coloring as default if we are using petscASM.#5211
Molin-Han wants to merge 5 commits into
mainfrom
molin/ASMColoring

Conversation

@Molin-Han

Copy link
Copy Markdown
Contributor

Description

In ASMStarPC and ASMVankaPC, make the coloring feature as the default option if we are using 'petscasm'. If we are using tinyasm, we do not use the coloring.

@Molin-Han Molin-Han requested a review from pbrubeck June 30, 2026 13:40
Comment thread firedrake/preconditioners/asm.py Outdated
Comment thread firedrake/preconditioners/asm.py Outdated
@pbrubeck

Copy link
Copy Markdown
Contributor

We should store self._backend in the base class instead of grabbing the option in multiple places.

Molin-Han and others added 3 commits June 30, 2026 16:39
Comment thread firedrake/preconditioners/asm.py Outdated
Comment thread firedrake/preconditioners/asm.py
Co-authored-by: Pablo Brubeck <brubeck@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants