Skip to content
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

Deprecate SuperLU #518

Closed
spdomin opened this issue Jan 5, 2021 · 3 comments
Closed

Deprecate SuperLU #518

spdomin opened this issue Jan 5, 2021 · 3 comments
Assignees

Comments

@spdomin
Copy link
Contributor

spdomin commented Jan 5, 2021

SuperLU is only used for a coarse direct solve for Muelu's AMG approach. As such, the standard KLU build in Trilinos works perfectly fine.

Lets:

  1. Deprecate SuperLU
  2. Modify the manual build instructions to say that SuperLU is optional with KLU available if the Muelu XML file includes "KLU" (for a Trilinos install that includes SuperLU).
  3. Accept the diffs for gcc 8.3/mpi4.0.3
@spdomin
Copy link
Contributor Author

spdomin commented Jan 5, 2021

This is the Mulue XML note that we should add if a user has a SuperLU Trilinos install, however, wants to use KLU:

<Parameter name="coarse: type" type="string" value="klu2"/>

@spdomin
Copy link
Contributor Author

spdomin commented Jan 5, 2021

Basically, the premise here is that if a Nalu build used a version of Trilinos that did not include SuperLU, then KLU is the automatic default.

@spdomin
Copy link
Contributor Author

spdomin commented Jan 12, 2021

Done - document is up-to-date as well.

170349b

@spdomin spdomin closed this as completed Jan 12, 2021
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

No branches or pull requests

2 participants