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

Added 2015 Illapel earthquake example #4

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

apadaki
Copy link

@apadaki apadaki commented Aug 22, 2021

No description provided.

@mandli
Copy link
Member

mandli commented Aug 22, 2021

@apadaki This looks good to me. @rjleveque do you want to take a look at this to make sure this looks good and does not duplicate anything that might be somewhere I do not know about?

@rjleveque
Copy link
Member

@apadaki, thanks for doing this! It's nice that you download the USGS fault parameters and the gauge observations directly in the maketopo.py script.

As set up, the code takes quite a while to run and doesn't seem to be refining optimally for capturing the signal at the two gauges, e.g. the refinement of the main wave is mostly to the south of gauge 32412. I think it would be better if you could add some flagregions (preferably in the new style) to guide the refinement more, allowing high levels only on the path towards the gauges for this example.

It might also run faster if you use fewer levels with larger refinement ratios, I notice some levels only refine by 2, you could have fewer levels always refining by 4, for example. And maybe you don't need such fine levels in this example? People can add more if they want to, but unless such fine grids are needed to illustrate something, it might be nicer if it runs faster by default.

@apadaki
Copy link
Author

apadaki commented Aug 24, 2021

@rjleveque Thank you for your feedback! I'll make sure to implement fewer refinement levels to improve the run time and specify manual regions in a commit sometime today or tomorrow.

@apadaki
Copy link
Author

apadaki commented Aug 25, 2021

@rjleveque I simplified the AMR ratios and added flagregions around each gauge, and the simulation seems to run much faster and give similar results on both gauges. If anything else needs to be changed, please let me know!

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.

3 participants