Skip to content

Commit 3e65c40

Browse files
authored
Fix accuracy test create PR (#2274)
### What this PR does / why we need it? Fix create PR of accuracy test ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? Local testing: nv-action/vllm-benchmarks#87 - vLLM version: v0.10.0 - vLLM main: vllm-project/vllm@099c046 --------- Signed-off-by: Icey <[email protected]>
1 parent 0bd5ff5 commit 3e65c40

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/accuracy_test.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -226,14 +226,14 @@ jobs:
226226

227227
outputs:
228228
model_name: ${{ steps.set_output.outputs.model_name }}
229-
229+
vllm_ascend_version: ${{ env.GHA_VLLM_ASCEND_VERSION }}
230+
230231
create_pr:
231232
runs-on: ubuntu-latest
232233
needs: accuracy_tests
233234
if: ${{ github.event_name == 'workflow_dispatch' && github.event.inputs.vllm-ascend-version == 'latest' }}
234235
env:
235236
UPSTREAM_REPO: vllm-project/vllm-ascend
236-
237237
steps:
238238
- name: Checkout repository
239239
uses: actions/checkout@v4
@@ -258,7 +258,7 @@ jobs:
258258
TIMESTAMP=$(date +%Y%m%d%H%M%S)
259259
BRANCH_NAME="auto-pr/accuracy-report-${TIMESTAMP}"
260260
echo "BRANCH_NAME=${BRANCH_NAME}" >> $GITHUB_ENV
261-
git checkout -B "${BRANCH_NAME}" upstream/${{ github.event.inputs.vllm-ascend-version }}
261+
git checkout -B "${BRANCH_NAME}" upstream/main
262262
263263
- name: Download only current run reports
264264
uses: actions/download-artifact@v4
@@ -299,7 +299,7 @@ jobs:
299299
GITHUB_TOKEN: ${{ secrets.PAT_TOKEN }}
300300
run: |
301301
git add ./docs/source/developer_guide/evaluation/accuracy_report/*.md
302-
git commit -s -m "[Doc] Update accuracy reports for ${{ github.event.inputs.vllm-ascend-version }}"
302+
git commit -s -m "[Doc] Update accuracy reports for ${{ needs.accuracy_tests.outputs.vllm_ascend_version }}"
303303
git push -f origin "${{ env.BRANCH_NAME }}"
304304
305305
- name: Create PR in upstream via API
@@ -311,8 +311,8 @@ jobs:
311311
owner: 'vllm-project',
312312
repo: 'vllm-ascend',
313313
head: `vllm-ascend-ci:${{ env.BRANCH_NAME }}`,
314-
base: '${{ github.event.inputs.vllm-ascend-version }}',
315-
title: `[Doc] Update accuracy reports for ${{ github.event.inputs.vllm-ascend-version }}`,
314+
base: 'main',
315+
title: `[Doc] Update accuracy reports for ${{ needs.accuracy_tests.outputs.vllm_ascend_version }}`,
316316
body: `The accuracy results running on NPU Altlas A2 have changed, updating reports for: All models (Qwen3-30B-A3B, Qwen2.5-VL-7B-Instruct, Qwen3-8B-Base, DeepSeek-V2-Lite)
317317
318318
- [Workflow run][1]

0 commit comments

Comments
 (0)