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

ASPECT 3: Update some of our default values #5568

Open
6 tasks
jdannberg opened this issue Feb 7, 2024 · 1 comment
Open
6 tasks

ASPECT 3: Update some of our default values #5568

jdannberg opened this issue Feb 7, 2024 · 1 comment
Milestone

Comments

@jdannberg
Copy link
Contributor

jdannberg commented Feb 7, 2024

This came out of a discussion at the ASPECT User Meeting. There are some default values that we have found out later could be chosen in a better way and that we might want to update. Let's collect all of those in this issue:

  • Change GMG to be the default solver [WIP] Make the GMG solver the new default solver. #5211
  • For GMG: Increase number of Cheap solver steps/do not do expensive steps at all
  • Particles: Increase Number of particles per cell per direction for Reference cell generator
  • Particles: Is cell average the right default interpolator?
  • Particles: Increase Minimum particles per cell?
  • Should the default be to do no adaptive refinement? (Refinement + Coarsening fraction, Time steps between mesh refinement)
@jdannberg jdannberg added this to the 3.0 milestone Feb 7, 2024
@tjhei
Copy link
Member

tjhei commented Feb 7, 2024

  • For GMG: Increase number of Cheap solver steps/do not do expensive steps at all

I would generally recommend choosing 0 expensive steps for GMG. That is different for the AMG solver, though.

With the current situation of the GMG not converging when switching from cheap to expensive, we could enforce either of the two values to be 0. @MFraters does that make sense to you?

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