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

Variorium Connector: kernel granularity and JSON #265

Merged
merged 33 commits into from
Jan 17, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
2fb4b91
first commit number 3
Jul 15, 2024
554b7f9
variorum updates
Jul 19, 2024
c4e7b0a
variorum updates
Jul 19, 2024
4e88454
clang update
Jul 22, 2024
1abd450
updated based on concerns
Aug 5, 2024
0c5d95f
Merge branch 'kokkos:develop' into varconnect
twilk10 Aug 6, 2024
e9971f8
remove build directory
Aug 6, 2024
69c8a4c
varconnect deleted
Aug 6, 2024
6c8a4c7
remove unneeded files
Aug 6, 2024
1932e24
addressing comments on pull request
Aug 7, 2024
090eda4
addressing comments on pull request
Aug 7, 2024
803f61f
Update profiling/variorum-connector/variorum-connector.cpp
twilk10 Aug 8, 2024
eebc9f4
update based on comments on pull request 10:48 8/8
Aug 8, 2024
4aefd92
updates based off of pull requests 15:37 8/8
Aug 8, 2024
56ac9cd
updates based off of pull request comments 14:04 8/9
Aug 9, 2024
7e33048
updates based off of pull request comments 14:20 8/9
Aug 9, 2024
f4a765d
Clean up the variorum connector some more
masterleinad Aug 19, 2024
357b8a2
Remove another unused variable
masterleinad Oct 14, 2024
94c8817
Replace fprintf
masterleinad Jan 16, 2025
083d345
Don;t call std:string constructor unnecessarily
masterleinad Jan 16, 2025
45c1620
Build variorum
masterleinad Jan 16, 2025
75e4bfd
Simplify
masterleinad Jan 16, 2025
281914c
Fix formatting
masterleinad Jan 16, 2025
b915c2f
Install and set Fortran compiler
masterleinad Jan 16, 2025
eff8334
Install Jansson
masterleinad Jan 16, 2025
c765e50
Simplify installation
masterleinad Jan 16, 2025
b117cf8
Variorum_ROOT -> VARIORUM_ROOT
masterleinad Jan 16, 2025
40dae3d
Don't set CMAKE_PREFIX_PATH
masterleinad Jan 17, 2025
d0807bc
Fix typo
masterleinad Jan 17, 2025
5f80c3d
Skip Variorum test
masterleinad Jan 17, 2025
e8468da
Don't use GTEST_FILTER
masterleinad Jan 17, 2025
37aab14
Improve comment
masterleinad Jan 17, 2025
d0e4f6c
Improve comment again
masterleinad Jan 17, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 21 additions & 4 deletions .github/workflows/build-with-kokkos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
image: ${{ matrix.image }}
env:
Kokkos_ROOT: /opt/kokkos
VARIORUM_ROOT: /opt/variorum
steps:
- name: Checkout Kokkos Tools
uses: actions/checkout@v4
Expand All @@ -38,22 +39,29 @@ jobs:
repository: kokkos/kokkos
path: kokkos
ref: develop
- name: Checkout Varioum
uses: actions/checkout@v4
with:
repository: LLNL/variorum
path: variorum
ref: 249a39bf41972afe2213d3dc904e47647364f466
- name: Install compilers
run : |
apt update

if [ ${{ matrix.compiler }} != 'default' ];then
apt --yes --no-install-recommends install ${{ matrix.compiler.c }} ${{ matrix.compiler.cpp }}
apt --yes --no-install-recommends install ${{ matrix.compiler.c }} ${{ matrix.compiler.cpp }} gfortran
export CC=${{ matrix.compiler.c }}
export CXX=${{ matrix.compiler.cpp }}
else
apt --yes --no-install-recommends install gcc g++
apt --yes --no-install-recommends install gcc g++ gfortran
export CC=gcc
export CXX=g++
fi

echo "CC=$CC" >> $GITHUB_ENV
echo "CXX=$CXX" >> $GITHUB_ENV
echo "FC=gfortran" >> $GITHUB_ENV

case ${{ matrix.preset }} in
*OpenMP* )
Expand All @@ -71,14 +79,22 @@ jobs:
exit -1
esac

- name: Install git, CMake, OpenMPI, PAPI and dtrace
- name: Install git, CMake, Jansson, OpenMPI, PAPI and dtrace
run: |
apt --yes --no-install-recommends install \
git ca-certificates \
cmake make \
libopenmpi-dev \
systemtap-sdt-dev \
libjansson-dev \
libpapi-dev
- name: Compile and install Variorum
working-directory: variorum
run: |
cp ${GITHUB_WORKSPACE}/kokkos.presets.json CMakePresets.json
cmake -S src -B build -DCMAKE_INSTALL_PREFIX=${VARIORUM_ROOT}
cd build
make install
- name: Compile and install Kokkos
working-directory: kokkos
run: |
Expand All @@ -92,7 +108,8 @@ jobs:
cmake --build --preset=${{ matrix.preset }}
cmake --install build-with-${{ matrix.preset }} --prefix=/opt/kokkos-tools
# For now, GitHub runners are used. These runner don't have GPUs. Therefore, testing can only be done for OpenMP.
# Skip variorum test since the architecture detected by variorum is not supported
- name: Run tests
if: ${{ matrix.preset == 'OpenMP' }}
run: |
ctest --preset=${{ matrix.preset }}
ctest -E "test_kokkos_tools_variorum" --preset=${{ matrix.preset }}
dalg24 marked this conversation as resolved.
Show resolved Hide resolved
Loading
Loading