-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update moose-mpi & PETSc: default Python packages, pinnings, apptainer exec fix, optimize compiler flags #28656
base: next
Are you sure you want to change the base?
Conversation
0a2a55f
to
6521d79
Compare
6521d79
to
e031df6
Compare
e031df6
to
ecd0a29
Compare
Remove DEBUG and other offending flags from CXXFLAGS/CPPFLAGS. When installing moose-mpi, supply a default set of python packages necessary to build MOOSE. Continue to constrain to a Python range. Default Python is latest supported version (3.11 at this commit) Pin HDF5 to 1.14.2 due to discovered issue: idaholab#28350 As requested, incorporating `set` options we require when launching containers with exec subcommand. Bump SDK for Mac Intel (PETSc contrib requirement). Transition to files section for exec fix Pin several packages, as we are tired of experiencing issues with only small bumps to build versions. Refs idaholab#28427 idaholab#28283 Closes idaholab#28643
ecd0a29
to
7de692b
Compare
GPERFTOOLS_SRC=${ROOT_BUILD_DIR}/gperftools | ||
cd ${GPERFTOOLS_SRC} | ||
./autogen.sh | ||
./configure --prefix={{ GPERF_DIR }} --enable-frame-pointers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are people hoping to do a lot of heap profiling? That would be the reason I'd recommend going to the frame-pointer route. Otherwise using libunwind gives more accurate stack traces, particularly in my experience if fortran libraries like mumps are involved
I'm confused. Is this meant to replace #28587 ? |
It combines it into this one. No need to update mpi and up, and then petsc and up in two PRs. |
Job Documentation on 7de692b wanted to post the following: View the site here This comment will be updated on new commits. |
Job Coverage on 7de692b wanted to post the following: Framework coverageCoverage did not change Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
Remove DEBUG and other offending flags from CXXFLAGS/CPPFLAGS.
When installing moose-mpi, supply a default set of python packages necessary to build MOOSE.
Continue to constrain to a Python range. Default Python is latest supported version (3.11 at this commit)
Pin HDF5 to 1.14.2 due to discovered issue: #28350
As requested, incorporating
set
options we require when launching containers with exec subcommand.Bump SDK for Mac Intel (PETSc contrib requirement).
Transition to files section for exec fix
Pin several packages, as we are tired of experiencing issues with only small bumps to build versions.
Refs #28427 #28283
Closes #28643