Skip to content

Commit 2b9291c

Browse files
authored
[CI] Fix defaults values in benchmark CI (#18072)
This PR: - Fixes missing default values for benchmarking CI manual trigger - When splitting conditionals into multiple lines, if there is a newline at the end of the expression the github yml appends a newline if you use `|` or `>` (which, by definition, should have gotten rid of the newline): This PR folds the conditional expression back into 1 line
1 parent ffc8247 commit 2b9291c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/sycl-ur-perf-benchmarking.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,8 +114,8 @@ jobs:
114114
strategy:
115115
matrix:
116116
include:
117-
- ref: ${{ inputs.commit_hash != '' && inputs.commit_hash || format('refs/pull/{0}/head', inputs.pr_no) }}
118-
save_name: ${{ inputs.commit_hash != '' && format('Commit{0}', inputs.commit_hash) || format('PR{0}', inputs.pr_no) }}
117+
- ref: ${{ inputs.commit_hash != '' && inputs.commit_hash || inputs.pr_no != '' && format('refs/pull/{0}/head', inputs.pr_no) || github.ref }}
118+
save_name: ${{ inputs.commit_hash != '' && format('Commit{0}', inputs.commit_hash) || inputs.pr_no != '' && format('PR{0}', inputs.pr_no) || 'Baseline' }}
119119
# Set default values if not specified:
120120
runner: ${{ inputs.runner || '["PVC_PERF"]' }}
121121
backend: ${{ inputs.backend || 'level_zero:gpu' }}

0 commit comments

Comments
 (0)