Skip to content

Commit

Permalink
fix MUSIC setup.py for multiple MPI include dirs
Browse files Browse the repository at this point in the history
add coverage setup
  • Loading branch information
alexsavulescu committed Feb 6, 2023
1 parent 7ca3f54 commit a15d446
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
18 changes: 17 additions & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,22 @@ jobs:
cmake ..
make && sudo make install
- name: Setup MUSIC
run: |
python3 -m venv music-venv
source music-venv/bin/activate
python3 -m pip install mpi4py cython numpy
# Stable build: https://github.com/INCF/MUSIC/archive/refs/heads/switch-to-MPI-C-interface.zip @ f33b66ea9348888eed1761738ab48c23ffc8a0d0
curl -L -o MUSIC.zip https://codeload.github.com/INCF/MUSIC/zip/f33b66ea9348888eed1761738ab48c23ffc8a0d0 &&
unzip MUSIC.zip &&
mv MUSIC-* MUSIC &&
cd MUSIC &&
./autogen.sh &&
./configure --with-python-sys-prefix --disable-anysource &&
sudo make -j install &&
cd .. && sudo rm -rf MUSIC*
deactivate
- name: Setup Xvfb
run: |
sudo apt-get install xvfb
Expand Down Expand Up @@ -97,7 +113,7 @@ jobs:
# CMake options & flags
export COVERAGE_FLAGS="--coverage -O0 -fno-inline -g";
export CMAKE_OPTION="-DNRN_ENABLE_MPI=ON -DNRN_ENABLE_INTERVIEWS=ON -DNRN_ENABLE_PYTHON=ON -DNRN_ENABLE_PYTHON_DYNAMIC=ON -DNRN_PYTHON_DYNAMIC=${PYTHON_MIN};${PYTHON_MAX} -DNRN_ENABLE_CORENEURON=ON -DNRN_ENABLE_PROFILING=ON -DNRN_ENABLE_BACKTRACE=ON";
export CMAKE_OPTION="-DNRN_ENABLE_MPI=ON -DNRN_ENABLE_INTERVIEWS=ON -DNRN_ENABLE_PYTHON=ON -DNRN_ENABLE_PYTHON_DYNAMIC=ON -DNRN_PYTHON_DYNAMIC=${PYTHON_MIN};${PYTHON_MAX} -DNRN_ENABLE_CORENEURON=ON -DNRN_ENABLE_PROFILING=ON -DNRN_ENABLE_BACKTRACE=ON -DNRN_ENABLE_MUSIC=ON";
cmake $CMAKE_OPTION -DCMAKE_C_COMPILER=$CC -DCMAKE_CXX_COMPILER=$CXX -DNRN_ENABLE_TESTS=ON -DCMAKE_C_FLAGS="${COVERAGE_FLAGS}" -DCMAKE_CXX_FLAGS="${COVERAGE_FLAGS}" -DCORENRN_ENABLE_UNIT_TESTS=ON ..;
# Coverage
Expand Down
2 changes: 1 addition & 1 deletion src/neuronmusic/setup.py.in
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import os
os.environ["CC"]=mpicc_bin
os.environ["CXX"]=mpicxx_bin

include_dirs = ['@MUSIC_INCDIR@', '@MPI_INCLUDE_PATH@', nrn_srcdir+'/src/neuronmusic', nrn_srcdir + '/src/nrnpython', '.']
include_dirs = ['@MUSIC_INCDIR@', nrn_srcdir+'/src/neuronmusic', nrn_srcdir + '/src/nrnpython', '.'] + '@MPI_INCLUDE_PATH@'.split(';')
libdirs = ['@MUSIC_LIBDIR@', @NRN_LIBDIR@]


Expand Down

0 comments on commit a15d446

Please sign in to comment.