Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion .github/workflows/sycl-containers-igc-dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ permissions: read-all

jobs:
build_and_push_images:
if: github.repository == 'intel/llvm'
if: false
name: Build and Push IGC Dev Docker Images
runs-on: ubuntu-latest
permissions:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/sycl-linux-precommit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ permissions: read-all

jobs:
detect_changes:
if: false
uses: ./.github/workflows/sycl-detect-changes.yml

build:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/sycl-windows-precommit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ concurrency:

jobs:
detect_changes:
if: false
uses: ./.github/workflows/sycl-detect-changes.yml

build:
Expand Down
113 changes: 3 additions & 110 deletions .github/workflows/ur-benchmarks-reusable.yml
Original file line number Diff line number Diff line change
@@ -1,32 +1,7 @@
name: Benchmarks Reusable

on:
workflow_call:
inputs:
str_name:
required: true
type: string
pr_no:
required: true
# even though this is a number, this is a workaround for issues with
# reusable workflow calls that result in "Unexpected value '0'" error.
type: string
bench_script_params:
required: false
type: string
default: ''
sycl_config_params:
required: false
type: string
default: ''
upload_report:
required: false
type: boolean
default: false
compute_runtime_commit:
required: false
type: string
default: ''
pull_request:

permissions:
contents: read
Expand All @@ -38,8 +13,8 @@ jobs:
strategy:
matrix:
adapter: [
{str_name: "${{ inputs.str_name }}",
sycl_config: "${{ inputs.sycl_config_params }}"
{str_name: "level_zero",
sycl_config: ""
}
]
build_type: [Release]
Expand All @@ -48,38 +23,11 @@ jobs:
runs-on: "PVC_PERF"

steps:
- name: Add comment to PR
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
if: ${{ always() && inputs.pr_no != 0 }}
with:
script: |
const pr_no = '${{ inputs.pr_no }}';
const adapter = '${{ matrix.adapter.str_name }}';
const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
const params = '${{ inputs.bench_script_params }}';
const body = `Compute Benchmarks ${adapter} run (with params: ${params}):\n${url}`;

github.rest.issues.createComment({
issue_number: pr_no,
owner: context.repo.owner,
repo: context.repo.repo,
body: body
})

- name: Checkout SYCL
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
with:
path: sycl-repo

# We need to fetch special ref for proper PR's merge commit. Note, this ref may be absent if the PR is already merged.
- name: Fetch PR's merge commit
if: ${{ inputs.pr_no != 0 }}
working-directory: ${{github.workspace}}/sycl-repo
run: |
git fetch -- https://github.com/${{github.repository}} +refs/pull/${{ inputs.pr_no }}/*:refs/remotes/origin/pr/${{ inputs.pr_no }}/*
git checkout origin/pr/${{ inputs.pr_no }}/merge
git rev-parse origin/pr/${{ inputs.pr_no }}/merge

# TODO: As long as we didn't merge this workflow into main, we should allow both scripts location
- name: Establish bench scripts location
run: |
Expand Down Expand Up @@ -165,58 +113,3 @@ jobs:
--compare baseline
--compute-runtime ${{ inputs.compute_runtime_commit }}
--build-igc
${{ inputs.upload_report && '--output-html' || '' }}
${{ inputs.pr_no != 0 && '--output-markdown' || '' }}
${{ inputs.bench_script_params }}
# Temporarily disabled due to build faiures
# https://github.com/intel/llvm/actions/runs/13814877162/job/38645384849#step:14:849
# --ur ${{ github.workspace }}/ur_install

- name: Print benchmark results
run: |
cat ${{ github.workspace }}/benchmark_results.md || true

- name: Add comment to PR
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
if: ${{ always() && inputs.pr_no != 0 }}
with:
script: |
let markdown = ""
try {
const fs = require('fs');
markdown = fs.readFileSync('benchmark_results.md', 'utf8');
} catch(err) {
}

const pr_no = '${{ inputs.pr_no }}';
const adapter = '${{ matrix.adapter.str_name }}';
const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
const test_status = '${{ steps.benchmarks.outcome }}';
const job_status = '${{ job.status }}';
const params = '${{ inputs.bench_script_params }}';
const body = `Benchmarks ${adapter} run (${params}):\n${url}\nJob status: ${job_status}. Test status: ${test_status}.\n ${markdown}`;

github.rest.issues.createComment({
issue_number: pr_no,
owner: context.repo.owner,
repo: context.repo.repo,
body: body
})

- name: Rename benchmark results file
if: ${{ always() && inputs.upload_report }}
run: mv benchmark_results.html benchmark_results_${{ inputs.pr_no }}.html

- name: Upload HTML report
if: ${{ always() && inputs.upload_report }}
uses: actions/cache/save@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0
with:
path: benchmark_results_${{ inputs.pr_no }}.html
key: benchmark-results-${{ inputs.pr_no }}-${{ matrix.adapter.str_name }}-${{ github.run_id }}

# TODO: As long as we didn't merge this workflow into main, we should allow both scripts location
- name: Get information about platform
if: ${{ always() }}
run: |
${{github.workspace}}/sycl-repo/devops/scripts/get_system_info.sh || true
${{github.workspace}}/sycl-repo/unified-runtime/.github/scripts/get_system_info.sh || true
1 change: 1 addition & 0 deletions .github/workflows/ur-precommit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ permissions: read-all

jobs:
detect_changes:
if: false
name: Detect Changes
uses: ./.github/workflows/sycl-detect-changes.yml

Expand Down
6 changes: 3 additions & 3 deletions devops/scripts/benchmarks/benches/compute.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,9 @@ def setup(self):

repo_path = git_clone(
self.directory,
"compute-benchmarks-repo",
"https://github.com/intel/compute-benchmarks.git",
"9369275026229b182bc4a555b73c2ec995a9e2b7",
"compute-benchmarks-luke-repo",
"https://github.com/lukaszstolarczuk/compute-benchmarks.git",
"7f6c486ff7fea4e1c1bcdfdb58b6301fa871d410",
)
build_path = create_build_path(self.directory, "compute-benchmarks-build")

Expand Down
Loading