Skip to content

Commit

Permalink
[benchmarks] fix umf suite after recent changes
Browse files Browse the repository at this point in the history
  • Loading branch information
pbalcer committed Dec 13, 2024
1 parent f9efc51 commit 569f4f2
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 6 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/benchmarks-reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -176,9 +176,7 @@ jobs:
- name: Build UMF
run: cmake --build ${{github.workspace}}/umf_build -j $(nproc)

- name: Run benchmarks
working-directory: ${{ github.workspace }}/ur-repo/
id: benchmarks
- name: Compute core range
run: |
# Compute the core range for the first NUMA node, skipping the first 4 cores.
# This is to avoid the first cores that the kernel is likely to schedule more work on.
Expand All @@ -190,7 +188,13 @@ jobs:
sub(/^0/, "4", b[1])
print b[1]
}')
taskset -c $CORES ${{ github.workspace }}/ur-repo/scripts/benchmarks/main.py
echo "CORES=$CORES" >> $GITHUB_ENV
- name: Run benchmarks
working-directory: ${{ github.workspace }}/ur-repo/
id: benchmarks
run: >
taskset -c ${{ env.CORES }} ${{ github.workspace }}/ur-repo/scripts/benchmarks/main.py
~/bench_workdir
--sycl ${{ github.workspace }}/sycl_build
--ur ${{ github.workspace }}/ur_install
Expand Down
4 changes: 2 additions & 2 deletions scripts/benchmarks/benches/umf.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,8 @@ def benchmarks(self) -> list[Benchmark]:

class ComputeUMFBenchmark(Benchmark):
def __init__(self, bench, name):
super().__init__(bench.directory, bench)

self.bench = bench
self.bench_name = name
self.oneapi = get_oneapi()
Expand All @@ -55,8 +57,6 @@ def __init__(self, bench, name):

self.col_statistics_time = None

super().__init__(bench.directory)

def bin_args(self) -> list[str]:
return []

Expand Down

0 comments on commit 569f4f2

Please sign in to comment.