Closed sseraj closed 1 year ago
Merging #310 (4ded19b) into main (e7ada3d) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #310 +/- ##
=======================================
Coverage 41.98% 41.98%
=======================================
Files 13 13
Lines 4001 4001
=======================================
Hits 1680 1680
Misses 2321 2321
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Purpose
I updated the Intel config files so that they work with Intel MPI and HPE MPI (used on NAS). The config files check for the
I_MPI_ROOT
environment variable to determine if Intel MPI is being used.In addition, I removed the following outdated config files:
config.LINUX_INTEL_SAFE.mk
because-O2
is generally safe with ifort and we have been testing this for several years nowconfig.OSX_GFORTRAN.mk
because this is untested, most likely broken, and we have the ARM Docker images nowold_architectures
folder because these files are 10+ years old and clutter the search results for the repoExpected time until merged
1 week
Type of change
Testing
I tested on one of our Intel MPI images and on NAS with HPE MPI.
Checklist
flake8
andblack
to make sure the Python code adheres to PEP-8 and is consistently formattedfprettify
or C/C++ code withclang-format
as applicable