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

WIP: updated settings for PETSc / MPI for Bouss codes #631

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

Conversation

rjleveque
Copy link
Member

Based on latest version of PETSc, with help from @BarrySmith. Still need to add some more updates that might be on a branch of his and/or from @mjberger.


# You may want to use a different version of petscMPIoptions
# This setting uses the version in this directory:
export PETSC_OPTIONS="-options_file $PWD/petscMPIoptions"

Choose a reason for hiding this comment

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

Will PWD always work when the executable may else where?

Maybe use the default of

export PETSC_OPTIONS="-options_file ${CLAW}/geoclaw/examples/bouss/petscMPIoptions

Copy link
Member Author

Choose a reason for hiding this comment

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

I was thinking the user would execute this file from within this directory, but I agree that it would be clearer to put in the full path so they see how to adjust it, and in case they want to execute it from their own example directory. Thanks!

@BarrySmith
Copy link

I don't know how to approve in GitHub pull request; looks ok to me.

@@ -10,33 +10,29 @@
# https://petsc.org/release/manualpages/PC/PCSetFromOptions/
Copy link
Member Author

Choose a reason for hiding this comment

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

@BarrySmith: Is there also a petsc webpage we should mention that discusses the mpi_* options? The mpi_ksp-* options are not discussed on the page https://petsc.org/release/manualpages/KSP/KSPSetFromOptions, for example.

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