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

Add a default local.cfg for a candi installation #5240

Merged
merged 1 commit into from Jul 10, 2023

Conversation

gassmoeller
Copy link
Member

Our local installation instructions use the default candi.cfg configuration file, which always tries to install opencascade, petsc and other packages, requiring a lot of time and causing more compilation complications than necessary. In addition it does not enable native optimizations. This PR provides a local.cfg file that is better suited for ASPECT use and speeds up the GMG solver significantly for most users (many are not aware of the impact of native optimizations). With this file we also have more control over which deal.II version to suggest for users, the default seems to be the development version of deal.II.

@@ -0,0 +1,9 @@
PACKAGES="load:dealii-prepare once:cmake once:astyle once:hdf5 once:p4est once:trilinos dealii"

DEAL_II_VERSION=v9.4.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9.5 is already out, though perhaps not yet advertised: https://github.com/dealii/dealii/releases How about we use that right away? Then people won't have to update in a year.

@@ -0,0 +1,9 @@
PACKAGES="load:dealii-prepare once:cmake once:astyle once:hdf5 once:p4est once:trilinos dealii"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the nonlinear solvers discussion, I will talk about SUNDIALS. One of the questions we'll have to discuss is whether we can add that to the prereqs -- my suggestion will be "yes", so perhaps add it here already now?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bangerth Why rely on SUNDIALS? NOX is part of Trilinos and of course already available without additional packages.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We didn't recommend people to use NOX in the past (I'll have to update the deal.II docs accordingly -- patch coming up). So if you followed our recommendations, it wouldn't have included NOX.

But (i) Trilinos also doesn't include time steppers, (ii) my sense is that KINSOL is actually better than NOX, and (iii) NOX is not actively developed, whereas SUNDIALS is.

Comment on lines 23 to 24
We require at least the packages <span-class="smallcaps">p4est</span>,
Trilinos, and finally deal.II. In addition there are some
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We require at least the packages <span-class="smallcaps">p4est</span>,
Trilinos, and finally deal.II. In addition there are some
We require at least the packages <span-class="smallcaps">p4est</span>,
Trilinos, and finally deal.II. We might require SUNDIALS in future versions. In addition there are some

@gassmoeller
Copy link
Member Author

I am ok with those changes. I have no experience with sundials and in particular not with how often it leads to compile problems. I just tried it and it compiled fine for me, so I guess that is a good omen.

@bangerth
Copy link
Contributor

It's a C library, so pretty straightforward. And also super fast to compile and install (unlike, for example, Trilinos).

@tjhei tjhei merged commit 58e2032 into geodynamics:main Jul 10, 2023
6 checks passed
@gassmoeller gassmoeller deleted the install_local branch July 10, 2023 18:50
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.

None yet

3 participants