From 805933e2d37a8d2ed357ae4decb12fd7a613414d Mon Sep 17 00:00:00 2001 From: hfadzxy Date: Mon, 26 May 2025 11:30:12 +0800 Subject: [PATCH 1/5] [Bugfix] Fix accuarcy test Signed-off-by: hfadzxy --- .github/workflows/accuracy_report.yaml | 2 +- .github/workflows/accuracy_test.yaml | 50 ++++++++++++-------------- benchmarks/scripts/run_accuracy.py | 12 +++---- 3 files changed, 29 insertions(+), 35 deletions(-) diff --git a/.github/workflows/accuracy_report.yaml b/.github/workflows/accuracy_report.yaml index 57960b71944..32c7fc390d7 100644 --- a/.github/workflows/accuracy_report.yaml +++ b/.github/workflows/accuracy_report.yaml @@ -128,7 +128,7 @@ jobs: uses: peter-evans/create-pull-request@v7 with: token: ${{ secrets.PR_TOKEN }} - base: ${{ github.ref_name }} + base: ${{ github.event.inputs.branch }} branch: auto-pr/accuracy-test commit-message: "Update accuracy report for ${{ github.event.inputs.branch }}" add-paths: ./docs/source/developer_guide/evaluation/accuracy_report/*.md diff --git a/.github/workflows/accuracy_test.yaml b/.github/workflows/accuracy_test.yaml index f5f8d32f75f..22e5e104352 100644 --- a/.github/workflows/accuracy_test.yaml +++ b/.github/workflows/accuracy_test.yaml @@ -21,15 +21,31 @@ on: workflow_dispatch: inputs: vllm-version: - description: 'what vllm version to accuracy test?' + description: 'vllm version:' required: true - type: string + type: choice + options: + - main + - v0.9.0 + - v0.8.5.post1 + - v0.8.5 + - v0.8.4 + - v0.7.3 vllm-ascend-version: - description: 'what vllm-ascend version to accuracy test?' + description: 'vllm-ascend version:' required: true - type: string + type: choice + options: + - main + - v0.7.3-dev + - v0.7.3 + - v0.8.5rc1 + - v0.8.4rc2 + - v0.8.4rc1 + - v0.7.3rc2 + - v0.7.3rc1 models: - description: 'choose model(all/Qwen2.5-7B-Instruct/Llama-3.1-8B-Instruct/Qwen2.5-VL-7B-Instruct/Qwen3-8B-Base)' + description: 'model:' required: true type: choice options: @@ -111,37 +127,15 @@ jobs: ref: ${{ github.event.inputs.vllm-ascend-version }} fetch-depth: 0 - - name: Install pta - run: | - if [ ! -d /root/.cache/pta ]; then - mkdir -p /root/.cache/pta - fi - if [ ! -f /root/.cache/pta/torch_npu-2.5.1.dev20250320-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.whl ]; then - cd /root/.cache/pta - rm -rf pytorch_v2.5.1_py310* - wget https://pytorch-package.obs.cn-north-4.myhuaweicloud.com/pta/Daily/v2.5.1/20250320.3/pytorch_v2.5.1_py310.tar.gz - tar -zxvf pytorch_v2.5.1_py310.tar.gz - fi - pip install /root/.cache/pta/torch_npu-2.5.1.dev20250320-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.whl - - name: Install vllm-project/vllm-ascend working-directory: ./vllm-ascend run: | pip install -r requirements-dev.txt pip install -e . - - name: Checkout EleutherAI/lm-evaluation-harness repo - uses: actions/checkout@v4 - with: - repository: EleutherAI/lm-evaluation-harness - path: ./lm-eval - fetch-depth: 0 - - name: Install EleutherAI/lm-evaluation-harness - working-directory: ./lm-eval run: | - pip install -e . - pip install ray datasets==2.16.0 + pip install lm-eval ray datasets==2.16.0 - name: Collect version info run: | diff --git a/benchmarks/scripts/run_accuracy.py b/benchmarks/scripts/run_accuracy.py index 18579d64ec0..28b6784a813 100644 --- a/benchmarks/scripts/run_accuracy.py +++ b/benchmarks/scripts/run_accuracy.py @@ -38,22 +38,22 @@ MODEL_RUN_INFO = { "Qwen/Qwen2.5-7B-Instruct": - ("export MODEL_AEGS='{model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" + ("export MODEL_ARGS='pretrained={model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" "lm_eval --model vllm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" ), - "LLM-Research/Meta-Llama-3.1-8B-Instruct": - ("export MODEL_AEGS='{model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" + "meta-llama/Llama-3.1-8B-Instruct": + ("export MODEL_ARGS='pretrained={model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" "lm_eval --model vllm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" ), - "Qwen/Qwen3-8B": - ("export MODEL_AEGS='{model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" + "Qwen/Qwen3-8B-Base": + ("export MODEL_ARGS='pretrained={model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" "lm_eval --model vllm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" ), "Qwen/Qwen2.5-VL-7B-Instruct": - ("export MODEL_AEGS='{model}, max_model_len=8192,dtype=auto,tensor_parallel_size=2,max_images=2'\n" + ("export MODEL_ARGS='pretrained={model}, max_model_len=8192,dtype=auto,tensor_parallel_size=2,max_images=2'\n" "lm_eval --model vllm-vlm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --batch_size 1"), } From 91f102b0102b3742ff5a96cc55248ec35ed6e87c Mon Sep 17 00:00:00 2001 From: Yikun Jiang Date: Sun, 1 Jun 2025 00:47:34 +0800 Subject: [PATCH 2/5] fix Signed-off-by: Yikun Jiang --- .github/workflows/accuracy_test.yaml | 64 +++++++++++++++---------- .github/workflows/vllm_ascend_test.yaml | 1 - 2 files changed, 40 insertions(+), 25 deletions(-) diff --git a/.github/workflows/accuracy_test.yaml b/.github/workflows/accuracy_test.yaml index 22e5e104352..ffb7901684c 100644 --- a/.github/workflows/accuracy_test.yaml +++ b/.github/workflows/accuracy_test.yaml @@ -15,9 +15,11 @@ # This file is a part of the vllm-ascend project. # -name: Accuracy Tests +name: Benchmarks / accuracy on: + pull_request: + types: [ labeled ] workflow_dispatch: inputs: vllm-version: @@ -26,10 +28,8 @@ on: type: choice options: - main + - v0.9.0.1 - v0.9.0 - - v0.8.5.post1 - - v0.8.5 - - v0.8.4 - v0.7.3 vllm-ascend-version: description: 'vllm-ascend version:' @@ -38,12 +38,6 @@ on: options: - main - v0.7.3-dev - - v0.7.3 - - v0.8.5rc1 - - v0.8.4rc2 - - v0.8.4rc1 - - v0.7.3rc2 - - v0.7.3rc1 models: description: 'model:' required: true @@ -51,9 +45,7 @@ on: options: - all - Qwen/Qwen2.5-7B-Instruct - - meta-llama/Llama-3.1-8B-Instruct - Qwen/Qwen2.5-VL-7B-Instruct - - Qwen/Qwen3-8B-Base default: 'all' # Bash shells do not use ~/.profile or ~/.bashrc so these shells need to be explicitly @@ -65,16 +57,31 @@ defaults: jobs: model_tests: + # test will be triggered when tag '*-accuracy-test' & 'ready-for-test' or schedule job + if: >- + ${{ + (contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') || + contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test')) && + contains(github.event.pull_request.labels.*.name, 'ready-for-test') || + github.event_name == 'workflow_dispatch' + }} name: Model Test - ${{ matrix.model_name }} runs-on: 'linux-arm64-npu-2' strategy: matrix: + # the accuracy test will run: + # 1. workflow_dispatch with models input + # - all: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct, Qwen/Qwen3-8B-Base + # - specified but not all: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct, Qwen/Qwen3-8B-Base + # 2. PR labeled with "*-accuracy-test" + # - dense-accuracy-test: Qwen/Qwen2.5-7B-Instruct + # - vl-accuracy-test: Qwen/Qwen2.5-VL-7B-Instruct include: ${{ fromJSON( - (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct","output_file":"Qwen2.5-7B-Instruct"},{"model_name":"meta-llama/Llama-3.1-8B-Instruct","output_file":"Llama-3.1-8B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct","output_file":"Qwen2.5-VL-7B-Instruct"}, {"model_name":"Qwen/Qwen3-8B-Base","output_file":"Qwen3-8B-Base"}]') || - (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct","output_file":"Qwen2.5-7B-Instruct"}]') || - (github.event.inputs.models == 'meta-llama/Llama-3.1-8B-Instruct' && '[{"model_name":"meta-llama/Llama-3.1-8B-Instruct","output_file":"Llama-3.1-8B-Instruct"}]') || - (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct","output_file":"Qwen2.5-VL-7B-Instruct"}]') || - (github.event.inputs.models == 'Qwen/Qwen3-8B-Base' && '[{"model_name":"Qwen/Qwen3-8B-Base","output_file":"Qwen3-8B-Base"}]') + (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"meta-llama/Llama-3.1-8B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}, {"model_name":"Qwen/Qwen3-8B-Base"}]') || + (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]') || + (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]') || + contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]' || + contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]' ) }} fail-fast: false @@ -84,6 +91,7 @@ jobs: HF_ENDPOINT: https://hf-mirror.com HF_TOKEN: ${{ secrets.HF_TOKEN }} DATASET_SOURCE: ModelScope + VLLM_USE_MODELSCOPE: True steps: - name: Checkout repository @@ -112,18 +120,20 @@ jobs: with: repository: vllm-project/vllm path: ./vllm-empty - ref: ${{ github.event.inputs.vllm-version }} + # Please also change this when bump matched version + ref: ${{ github.event.inputs.vllm-version || 'v0.9.0' }} - name: Install vllm-project/vllm from source working-directory: ./vllm-empty run: VLLM_TARGET_DEVICE=empty pip install -e . - - name: Checkout vllm-project/vllm-ascend repo uses: actions/checkout@v4 with: repository: vllm-project/vllm-ascend path: ./vllm-ascend + # 1. If version specified (work_dispatch), do specified branch test + # 2. If no version (labeled PR), do PR accuracy test ref: ${{ github.event.inputs.vllm-ascend-version }} fetch-depth: 0 @@ -133,7 +143,7 @@ jobs: pip install -r requirements-dev.txt pip install -e . - - name: Install EleutherAI/lm-evaluation-harness + - name: Install lm-eval, ray, and datasets run: | pip install lm-eval ray datasets==2.16.0 @@ -166,26 +176,32 @@ jobs: echo "vLLM: ${{ env.VLLM_VERSION }}" - name: Run Accuracy Test for V0 + id: report working-directory: ./benchmarks env: VLLM_USE_V1: 0 PYTORCH_NPU_ALLOC_CONF: max_split_size_mb:256 run: | + model_base_name=$(basename ${{ matrix.model_name }}) + echo "model_base_name=$model_base_name" + echo "model_base_name=$model_base_name" >> $GITHUB_OUTPUT mkdir -p ./accuracy/V0 + echo "test111" > ./accuracy/V0/${model_base_name}.md python ./scripts/run_accuracy.py \ --model "${{ matrix.model_name }}" \ - --output "./accuracy/V0/${{ matrix.output_file }}.md" \ - --vllm_ascend_version "${{ github.event.inputs.vllm-ascend-version }}" \ + --output "./accuracy/V0/${model_base_name}.md" \ + --vllm_ascend_version "${{ github.event.inputs.vllm-ascend-version || 'current' }}" \ --cann_version "${{ env.CANN_VERSION }}" \ --torch_npu_version "${{ env.TORCH_NPU_VERSION }}" \ --torch_version "${{ env.TORCH_VERSION }}" \ --vllm_version "${{ env.VLLM_VERSION }}" + cat ./accuracy/V0/${model_base_name}.md - name: Upload Report for V0 uses: actions/upload-artifact@v4 with: - name: "${{ github.event.inputs.vllm-ascend-version }}-${{ matrix.output_file }}-V0-report" - path: ./benchmarks/accuracy/V0/${{ matrix.output_file }}.md + name: "${{ github.event.inputs.vllm-ascend-version }}-${{ steps.report.outputs.model_base_name }}-V0-report" + path: ./benchmarks/accuracy/V0/${{ steps.report.outputs.model_base_name }}.md if-no-files-found: warn retention-days: 90 overwrite: true diff --git a/.github/workflows/vllm_ascend_test.yaml b/.github/workflows/vllm_ascend_test.yaml index 3f5738c8d03..5fdaee13cd7 100644 --- a/.github/workflows/vllm_ascend_test.yaml +++ b/.github/workflows/vllm_ascend_test.yaml @@ -27,7 +27,6 @@ on: paths: - '*.txt' - '**/*.py' - - '.github/workflows/vllm_ascend_test.yaml' - '!docs/**' - 'pytest.ini' - '!benchmarks/**' From 7b784f0832ecba33595a81a2f406028a76a22a1a Mon Sep 17 00:00:00 2001 From: Yikun Jiang Date: Sun, 1 Jun 2025 00:53:29 +0800 Subject: [PATCH 3/5] tmp Signed-off-by: Yikun Jiang --- .github/workflows/accuracy_test.yaml | 75 +++++++++++++++---------- .github/workflows/vllm_ascend_test.yaml | 1 + benchmarks/scripts/run_accuracy.py | 2 +- 3 files changed, 47 insertions(+), 31 deletions(-) diff --git a/.github/workflows/accuracy_test.yaml b/.github/workflows/accuracy_test.yaml index ffb7901684c..3c71ba211d8 100644 --- a/.github/workflows/accuracy_test.yaml +++ b/.github/workflows/accuracy_test.yaml @@ -15,6 +15,10 @@ # This file is a part of the vllm-ascend project. # +# This test will be triggered: +# 1. PR labeled with: '*accuracy-test' (ONLY 1 label valid) & 'ready-for-test' +# 2. workflow_dispatch with models input +# See detail rule in strategy.matrix note name: Benchmarks / accuracy on: @@ -26,6 +30,8 @@ on: description: 'vllm version:' required: true type: choice + # Please also update this when bump matched version + # Current supported vLLM versions options: - main - v0.9.0.1 @@ -46,6 +52,7 @@ on: - all - Qwen/Qwen2.5-7B-Instruct - Qwen/Qwen2.5-VL-7B-Instruct + - Qwen/Qwen3-8B-Base default: 'all' # Bash shells do not use ~/.profile or ~/.bashrc so these shells need to be explicitly @@ -56,16 +63,16 @@ defaults: shell: bash -el {0} jobs: - model_tests: - # test will be triggered when tag '*-accuracy-test' & 'ready-for-test' or schedule job + accuracy_tests: + # test will be triggered when tag '*-accuracy-test' & 'ready-for-test' or workflow_dispatch job if: >- ${{ - (contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') || + (contains(github.event.pull_request.labels.*.name, 'accuracy-test') || + contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') || contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test')) && contains(github.event.pull_request.labels.*.name, 'ready-for-test') || github.event_name == 'workflow_dispatch' }} - name: Model Test - ${{ matrix.model_name }} runs-on: 'linux-arm64-npu-2' strategy: matrix: @@ -74,24 +81,33 @@ jobs: # - all: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct, Qwen/Qwen3-8B-Base # - specified but not all: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct, Qwen/Qwen3-8B-Base # 2. PR labeled with "*-accuracy-test" + # - accuracy-test: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct # - dense-accuracy-test: Qwen/Qwen2.5-7B-Instruct # - vl-accuracy-test: Qwen/Qwen2.5-VL-7B-Instruct include: ${{ fromJSON( - (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"meta-llama/Llama-3.1-8B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}, {"model_name":"Qwen/Qwen3-8B-Base"}]') || + (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"},{"model_name":"Qwen/Qwen3-8B-Base"}]') || (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]') || (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]') || + (github.event.inputs.models == 'Qwen/Qwen3-8B-Base' && '[{"model_name":"Qwen/Qwen3-8B-Base"}]') || + contains(github.event.pull_request.labels.*.name, 'accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]' || contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]' || contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]' ) }} fail-fast: false - + name: ${{ matrix.model_name }} accuracy container: image: m.daocloud.io/quay.io/ascend/cann:8.1.rc1-910b-ubuntu22.04-py3.10 env: - HF_ENDPOINT: https://hf-mirror.com - HF_TOKEN: ${{ secrets.HF_TOKEN }} DATASET_SOURCE: ModelScope VLLM_USE_MODELSCOPE: True + # Please also update this when bump matched version + GHA_VLLM_VERSION: ${{ github.event.inputs.vllm-version || 'v0.9.0' }} + # 1. If version specified (work_dispatch), do specified branch accuracy test + # 2. If no version (labeled PR), do accuracy test by default ref: + # The branch, tag or SHA to checkout. When checking out the repository that + # triggered a workflow, this defaults to the reference or SHA for that event. + # Otherwise, uses the default branch. + GHA_VLLM_ASCEND_VERSION: ${{ github.event.inputs.vllm-ascend-version }} steps: - name: Checkout repository @@ -120,8 +136,7 @@ jobs: with: repository: vllm-project/vllm path: ./vllm-empty - # Please also change this when bump matched version - ref: ${{ github.event.inputs.vllm-version || 'v0.9.0' }} + ref: ${{ env.GHA_VLLM_VERSION }} - name: Install vllm-project/vllm from source working-directory: ./vllm-empty @@ -132,9 +147,7 @@ jobs: with: repository: vllm-project/vllm-ascend path: ./vllm-ascend - # 1. If version specified (work_dispatch), do specified branch test - # 2. If no version (labeled PR), do PR accuracy test - ref: ${{ github.event.inputs.vllm-ascend-version }} + ref: ${{ env.GHA_VLLM_ASCEND_VERSION }} fetch-depth: 0 - name: Install vllm-project/vllm-ascend @@ -157,23 +170,24 @@ jobs: fi done INFO_FILE="/usr/local/Ascend/ascend-toolkit/${TOOLKIT_DIR}/$(uname -i)-linux/ascend_toolkit_install.info" - CANN_VERSION=$(grep "version=" "$INFO_FILE" \ + GHA_CANN_VERSION=$(grep "version=" "$INFO_FILE" \ | head -n1 \ | cut -d'=' -f2 \ | tr -d '"') { - echo "CANN_VERSION=$CANN_VERSION" - pip show torch | grep "Version:" | awk '{print "TORCH_VERSION="$2}' - pip show torch_npu | grep "Version:" | awk '{print "TORCH_NPU_VERSION="$2}' - pip show vllm | grep "Version:" | awk '{print "VLLM_VERSION="$2}' | sed 's/+.*//' + echo "GHA_CANN_VERSION=$GHA_CANN_VERSION" + pip show torch | grep "Version:" | awk '{print "GHA_TORCH_VERSION="$2}' + pip show torch_npu | grep "Version:" | awk '{print "GHA_TORCH_NPU_VERSION="$2}' + pip show vllm | grep "Version:" | awk '{print "GHA_VLLM_VERSION="$2}' | sed 's/+.*//' } >> "$GITHUB_ENV" - name: Print versions run: | - echo "CANN: ${{ env.CANN_VERSION }}" - echo "Torch NPU: ${{ env.TORCH_NPU_VERSION }}" - echo "Torch: ${{ env.TORCH_VERSION }}" - echo "vLLM: ${{ env.VLLM_VERSION }}" + echo "CANN: ${{ env.GHA_CANN_VERSION }}" + echo "Torch NPU: ${{ env.GHA_TORCH_NPU_VERSION }}" + echo "Torch: ${{ env.GHA_TORCH_VERSION }}" + echo "vLLM: ${{ env.GHA_VLLM_VERSION }}" + echo "vLLM Ascend: ${{ env.GHA_VLLM_ASCEND_VERSION }}" - name: Run Accuracy Test for V0 id: report @@ -186,21 +200,22 @@ jobs: echo "model_base_name=$model_base_name" echo "model_base_name=$model_base_name" >> $GITHUB_OUTPUT mkdir -p ./accuracy/V0 - echo "test111" > ./accuracy/V0/${model_base_name}.md + python ./scripts/run_accuracy.py \ --model "${{ matrix.model_name }}" \ --output "./accuracy/V0/${model_base_name}.md" \ - --vllm_ascend_version "${{ github.event.inputs.vllm-ascend-version || 'current' }}" \ - --cann_version "${{ env.CANN_VERSION }}" \ - --torch_npu_version "${{ env.TORCH_NPU_VERSION }}" \ - --torch_version "${{ env.TORCH_VERSION }}" \ - --vllm_version "${{ env.VLLM_VERSION }}" - cat ./accuracy/V0/${model_base_name}.md + --vllm_ascend_version "${{ env.GHA_VLLM_ASCEND_VERSION || github.ref }}" \ + --cann_version "${{ env.GHA_CANN_VERSION }}" \ + --torch_npu_version "${{ env.GHA_TORCH_NPU_VERSION }}" \ + --torch_version "${{ env.GHA_TORCH_VERSION }}" \ + --vllm_version "${{ env.GHA_VLLM_VERSION }}" + + cat ./accuracy/V0/${model_base_name}.md >> $GITHUB_STEP_SUMMARY - name: Upload Report for V0 uses: actions/upload-artifact@v4 with: - name: "${{ github.event.inputs.vllm-ascend-version }}-${{ steps.report.outputs.model_base_name }}-V0-report" + name: "${{ env.GHA_VLLM_ASCEND_VERSION }}-${{ steps.report.outputs.model_base_name }}-V0-report" path: ./benchmarks/accuracy/V0/${{ steps.report.outputs.model_base_name }}.md if-no-files-found: warn retention-days: 90 diff --git a/.github/workflows/vllm_ascend_test.yaml b/.github/workflows/vllm_ascend_test.yaml index 5fdaee13cd7..3f5738c8d03 100644 --- a/.github/workflows/vllm_ascend_test.yaml +++ b/.github/workflows/vllm_ascend_test.yaml @@ -27,6 +27,7 @@ on: paths: - '*.txt' - '**/*.py' + - '.github/workflows/vllm_ascend_test.yaml' - '!docs/**' - 'pytest.ini' - '!benchmarks/**' diff --git a/benchmarks/scripts/run_accuracy.py b/benchmarks/scripts/run_accuracy.py index 28b6784a813..cd481c78e28 100644 --- a/benchmarks/scripts/run_accuracy.py +++ b/benchmarks/scripts/run_accuracy.py @@ -110,7 +110,7 @@ def generate_md(model_name, tasks_list, args, datasets): run_cmd = MODEL_RUN_INFO[model_name].format(model=model_name, datasets=datasets) model = model_name.split("/")[1] - preamble = f"""# {model} Accuracy Test + preamble = f"""# 🎯 {model} Accuracy Test
vLLM version: vLLM: {args.vllm_version}, vLLM Ascend: {args.vllm_ascend_version}
From 7d7790553674e46d4a292e3f29ec1449c5c73cd0 Mon Sep 17 00:00:00 2001 From: Yikun Jiang Date: Sun, 1 Jun 2025 21:12:20 +0800 Subject: [PATCH 4/5] vl -> tp4 Signed-off-by: Yikun Jiang --- .github/workflows/accuracy_test.yaml | 26 +++++++++++++------------- benchmarks/scripts/run_accuracy.py | 4 ++-- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/.github/workflows/accuracy_test.yaml b/.github/workflows/accuracy_test.yaml index 3c71ba211d8..5a4db9c1386 100644 --- a/.github/workflows/accuracy_test.yaml +++ b/.github/workflows/accuracy_test.yaml @@ -73,7 +73,7 @@ jobs: contains(github.event.pull_request.labels.*.name, 'ready-for-test') || github.event_name == 'workflow_dispatch' }} - runs-on: 'linux-arm64-npu-2' + runs-on: ${{ matrix.runner || 'linux-arm64-npu-2' }} strategy: matrix: # the accuracy test will run: @@ -85,23 +85,23 @@ jobs: # - dense-accuracy-test: Qwen/Qwen2.5-7B-Instruct # - vl-accuracy-test: Qwen/Qwen2.5-VL-7B-Instruct include: ${{ fromJSON( - (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"},{"model_name":"Qwen/Qwen3-8B-Base"}]') || + (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"},{"model_name":"Qwen/Qwen3-8B-Base"}]') || (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]') || - (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]') || + (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]') || (github.event.inputs.models == 'Qwen/Qwen3-8B-Base' && '[{"model_name":"Qwen/Qwen3-8B-Base"}]') || - contains(github.event.pull_request.labels.*.name, 'accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]' || + contains(github.event.pull_request.labels.*.name, 'accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]' || contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]' || - contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct"}]' + contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]' ) }} fail-fast: false name: ${{ matrix.model_name }} accuracy container: image: m.daocloud.io/quay.io/ascend/cann:8.1.rc1-910b-ubuntu22.04-py3.10 env: + HF_ENDPOINT: https://hf-mirror.com + HF_TOKEN: ${{ secrets.HF_TOKEN }} DATASET_SOURCE: ModelScope VLLM_USE_MODELSCOPE: True - # Please also update this when bump matched version - GHA_VLLM_VERSION: ${{ github.event.inputs.vllm-version || 'v0.9.0' }} # 1. If version specified (work_dispatch), do specified branch accuracy test # 2. If no version (labeled PR), do accuracy test by default ref: # The branch, tag or SHA to checkout. When checking out the repository that @@ -136,7 +136,8 @@ jobs: with: repository: vllm-project/vllm path: ./vllm-empty - ref: ${{ env.GHA_VLLM_VERSION }} + # Please also update this when bump matched version + ref: ${{ github.event.inputs.vllm-version || 'v0.9.0' }} - name: Install vllm-project/vllm from source working-directory: ./vllm-empty @@ -148,18 +149,17 @@ jobs: repository: vllm-project/vllm-ascend path: ./vllm-ascend ref: ${{ env.GHA_VLLM_ASCEND_VERSION }} - fetch-depth: 0 - name: Install vllm-project/vllm-ascend working-directory: ./vllm-ascend run: | pip install -r requirements-dev.txt pip install -e . - + - name: Install lm-eval, ray, and datasets run: | - pip install lm-eval ray datasets==2.16.0 - + pip install lm-eval + - name: Collect version info run: | for dir in /usr/local/Ascend/ascend-toolkit/*; do @@ -187,7 +187,7 @@ jobs: echo "Torch NPU: ${{ env.GHA_TORCH_NPU_VERSION }}" echo "Torch: ${{ env.GHA_TORCH_VERSION }}" echo "vLLM: ${{ env.GHA_VLLM_VERSION }}" - echo "vLLM Ascend: ${{ env.GHA_VLLM_ASCEND_VERSION }}" + echo "vLLM Ascend: ${{ env.GHA_VLLM_ASCEND_VERSION || github.ref }}" - name: Run Accuracy Test for V0 id: report diff --git a/benchmarks/scripts/run_accuracy.py b/benchmarks/scripts/run_accuracy.py index cd481c78e28..18c1de0c666 100644 --- a/benchmarks/scripts/run_accuracy.py +++ b/benchmarks/scripts/run_accuracy.py @@ -53,7 +53,7 @@ "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" ), "Qwen/Qwen2.5-VL-7B-Instruct": - ("export MODEL_ARGS='pretrained={model}, max_model_len=8192,dtype=auto,tensor_parallel_size=2,max_images=2'\n" + ("export MODEL_ARGS='pretrained={model}, max_model_len=8192,dtype=auto,tensor_parallel_size=4,max_images=2'\n" "lm_eval --model vllm-vlm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --batch_size 1"), } @@ -85,7 +85,7 @@ def run_accuracy_unimodal(queue, model, dataset): def run_accuracy_multimodal(queue, model, dataset): try: - model_args = f"pretrained={model},max_model_len=8192,dtype=auto,tensor_parallel_size=2,max_images=2" + model_args = f"pretrained={model},max_model_len=8192,dtype=auto,tensor_parallel_size=4,max_images=2" results = lm_eval.simple_evaluate( model="vllm-vlm", model_args=model_args, From dcf0bb92a9f0973dfab2e207d444b97b954d42dd Mon Sep 17 00:00:00 2001 From: Yikun Jiang Date: Mon, 2 Jun 2025 23:15:50 +0800 Subject: [PATCH 5/5] Suport V0 and V1 and remove unused mmlu Signed-off-by: Yikun Jiang --- .github/workflows/accuracy_report.yaml | 30 ++---------- .github/workflows/accuracy_test.yaml | 65 ++++++++++++++++++-------- benchmarks/scripts/run_accuracy.py | 15 ++---- 3 files changed, 54 insertions(+), 56 deletions(-) diff --git a/.github/workflows/accuracy_report.yaml b/.github/workflows/accuracy_report.yaml index 32c7fc390d7..1d03e131886 100644 --- a/.github/workflows/accuracy_report.yaml +++ b/.github/workflows/accuracy_report.yaml @@ -60,16 +60,6 @@ jobs: env: GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} - - name: Query artifact run id for Llama-3.1-8B-Instruct V0 latest artifact - id: get_Llama_3_1_8B_Instruct_latest_run_id_V0 - run: | - ARTIFACT_JSON=$(gh api "repos/${{ github.repository }}/actions/artifacts") - RUN_ID=$(echo "$ARTIFACT_JSON" | \ - jq -r '[.artifacts[] | select(.name=="${{ github.event.inputs.vllm-ascend-version }}-Llama-3.1-8B-Instruct-V0-report")] | sort_by(.created_at) | last | .workflow_run.id') - echo "runid=$RUN_ID" >> "$GITHUB_OUTPUT" - env: - GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} - - name: Query artifact run id for Qwen3-8B-Base V0 latest artifact id: get_Qwen3_8B_Base_latest_run_id_V0 run: | @@ -98,15 +88,6 @@ jobs: repository: vllm-project/vllm-ascend run-id: ${{ steps.get_Qwen2_5_7B_Instruct_latest_run_id_V0.outputs.runid }} - - name: Download meta-llama/Llama-3.1-8B-Instruct Artifact - uses: actions/download-artifact@v4 - with: - name: ${{ github.event.inputs.vllm-ascend-version }}-Llama-3.1-8B-Instruct-V0-report - path: ./docs/source/developer_guide/evaluation/accuracy_report - github-token: ${{ secrets.GITHUB_TOKEN }} - repository: vllm-project/vllm-ascend - run-id: ${{ steps.get_Llama_3_1_8B_Instruct_latest_run_id_V0.outputs.runid }} - - name: Download Qwen/Qwen3-8B-Base Artifact uses: actions/download-artifact@v4 with: @@ -120,7 +101,6 @@ jobs: working-directory: ./docs/source/developer_guide/evaluation/accuracy_report run: | cat ./Qwen2.5-VL-7B-Instruct.md - cat ./Llama-3.1-8B-Instruct.md cat ./Qwen2.5-7B-Instruct.md cat ./Qwen3-8B-Base.md @@ -139,12 +119,10 @@ jobs: - [Workflow run][1] - [Qwen2.5-7B-Instruct accuracy report][2] - - [Llama-3.1-8B-Instruct accuracy report][3] - - [Qwen2.5-VL-7B-Instruct accuracy report][4] - - [Qwen3-8B-Base accuracy report][5] + - [Qwen2.5-VL-7B-Instruct accuracy report][3] + - [Qwen3-8B-Base accuracy report][4] [1]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }} [2]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Qwen2_5_7B_Instruct_latest_run_id_V0.outputs.runid }} - [3]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Llama_3_1_8B_Instruct_latest_run_id_V0.outputs.runid }} - [4]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Qwen2_5_VL_7B_Instruct_latest_run_id_V0.outputs.runid }} - [5]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Qwen3_8B_Base_latest_run_id_V0.outputs.runid }} \ No newline at end of file + [3]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Qwen2_5_VL_7B_Instruct_latest_run_id_V0.outputs.runid }} + [4]: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ steps.get_Qwen3_8B_Base_latest_run_id_V0.outputs.runid }} diff --git a/.github/workflows/accuracy_test.yaml b/.github/workflows/accuracy_test.yaml index 5a4db9c1386..3bf6b746b0e 100644 --- a/.github/workflows/accuracy_test.yaml +++ b/.github/workflows/accuracy_test.yaml @@ -62,6 +62,10 @@ defaults: run: shell: bash -el {0} +concurrency: + group: pr-${{ github.event.pull_request.number }} + cancel-in-progress: true + jobs: accuracy_tests: # test will be triggered when tag '*-accuracy-test' & 'ready-for-test' or workflow_dispatch job @@ -73,9 +77,14 @@ jobs: contains(github.event.pull_request.labels.*.name, 'ready-for-test') || github.event_name == 'workflow_dispatch' }} - runs-on: ${{ matrix.runner || 'linux-arm64-npu-2' }} + runs-on: >- + ${{ + (matrix.model_name == 'Qwen/Qwen2.5-VL-7B-Instruct' && 'linux-arm64-npu-4') || + 'linux-arm64-npu-2' + }} strategy: matrix: + vllm_use_version: [0, 1] # the accuracy test will run: # 1. workflow_dispatch with models input # - all: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct, Qwen/Qwen3-8B-Base @@ -84,17 +93,29 @@ jobs: # - accuracy-test: Qwen/Qwen2.5-7B-Instruct, Qwen/Qwen2.5-VL-7B-Instruct # - dense-accuracy-test: Qwen/Qwen2.5-7B-Instruct # - vl-accuracy-test: Qwen/Qwen2.5-VL-7B-Instruct - include: ${{ fromJSON( - (github.event.inputs.models == 'all' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"},{"model_name":"Qwen/Qwen3-8B-Base"}]') || - (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]') || - (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]') || - (github.event.inputs.models == 'Qwen/Qwen3-8B-Base' && '[{"model_name":"Qwen/Qwen3-8B-Base"}]') || - contains(github.event.pull_request.labels.*.name, 'accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"},{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]' || - contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-7B-Instruct"}]' || - contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && '[{"model_name":"Qwen/Qwen2.5-VL-7B-Instruct", "runner":"linux-arm64-npu-4"}]' + model_name: ${{ fromJSON( + (github.event.inputs.models == 'all' && + '["Qwen/Qwen2.5-7B-Instruct","Qwen/Qwen2.5-VL-7B-Instruct","model_name":"Qwen/Qwen3-8B-Base"]') || + (github.event.inputs.models == 'Qwen/Qwen2.5-7B-Instruct' && + '["Qwen/Qwen2.5-7B-Instruct"]') || + (github.event.inputs.models == 'Qwen/Qwen2.5-VL-7B-Instruct' && + '["Qwen/Qwen2.5-VL-7B-Instruct"]') || + (github.event.inputs.models == 'Qwen/Qwen3-8B-Base' && + '["Qwen/Qwen3-8B-Base"]') || + contains(github.event.pull_request.labels.*.name, 'accuracy-test') && + '["Qwen/Qwen2.5-7B-Instruct","Qwen/Qwen2.5-VL-7B-Instruct"]' || + contains(github.event.pull_request.labels.*.name, 'dense-accuracy-test') && + '["Qwen/Qwen2.5-7B-Instruct"]' || + contains(github.event.pull_request.labels.*.name, 'vl-accuracy-test') && + '["Qwen/Qwen2.5-VL-7B-Instruct"]' ) }} + # Remove exclude after https://github.com/vllm-project/vllm-ascend/issues/1044 resolved + exclude: + - model_name: Qwen/Qwen2.5-VL-7B-Instruct + vllm_use_version: 1 + fail-fast: false - name: ${{ matrix.model_name }} accuracy + name: ${{ matrix.model_name }} accuracy V${{ matrix.vllm_use_version }} container: image: m.daocloud.io/quay.io/ascend/cann:8.1.rc1-910b-ubuntu22.04-py3.10 env: @@ -189,34 +210,38 @@ jobs: echo "vLLM: ${{ env.GHA_VLLM_VERSION }}" echo "vLLM Ascend: ${{ env.GHA_VLLM_ASCEND_VERSION || github.ref }}" - - name: Run Accuracy Test for V0 + - name: Run Accuracy Test for V${{ matrix.vllm_use_version }} id: report working-directory: ./benchmarks env: - VLLM_USE_V1: 0 PYTORCH_NPU_ALLOC_CONF: max_split_size_mb:256 + VLLM_USE_V1: ${{ matrix.vllm_use_version }} run: | model_base_name=$(basename ${{ matrix.model_name }}) - echo "model_base_name=$model_base_name" - echo "model_base_name=$model_base_name" >> $GITHUB_OUTPUT - mkdir -p ./accuracy/V0 + markdown_name="${model_base_name}-V${{ matrix.vllm_use_version }}" + echo "markdown_name=$markdown_name" + echo "markdown_name=$markdown_name" >> $GITHUB_OUTPUT + mkdir -p ./accuracy python ./scripts/run_accuracy.py \ --model "${{ matrix.model_name }}" \ - --output "./accuracy/V0/${model_base_name}.md" \ + --output "./accuracy/${markdown_name}.md" \ --vllm_ascend_version "${{ env.GHA_VLLM_ASCEND_VERSION || github.ref }}" \ --cann_version "${{ env.GHA_CANN_VERSION }}" \ --torch_npu_version "${{ env.GHA_TORCH_NPU_VERSION }}" \ --torch_version "${{ env.GHA_TORCH_VERSION }}" \ --vllm_version "${{ env.GHA_VLLM_VERSION }}" - cat ./accuracy/V0/${model_base_name}.md >> $GITHUB_STEP_SUMMARY + - name: Generate step summary + if: ${{ always() }} + run: | + cat ./benchmarks/accuracy/${{ steps.report.outputs.markdown_name }}.md >> $GITHUB_STEP_SUMMARY - - name: Upload Report for V0 + - name: Upload Report for V${{ matrix.vllm_use_version }} uses: actions/upload-artifact@v4 with: - name: "${{ env.GHA_VLLM_ASCEND_VERSION }}-${{ steps.report.outputs.model_base_name }}-V0-report" - path: ./benchmarks/accuracy/V0/${{ steps.report.outputs.model_base_name }}.md + name: "${{ env.GHA_VLLM_ASCEND_VERSION }}-${{ steps.report.outputs.markdown_name }}-report" + path: ./benchmarks/accuracy/${{ steps.report.outputs.markdown_name }}.md if-no-files-found: warn retention-days: 90 overwrite: true diff --git a/benchmarks/scripts/run_accuracy.py b/benchmarks/scripts/run_accuracy.py index 18c1de0c666..f508feda73c 100644 --- a/benchmarks/scripts/run_accuracy.py +++ b/benchmarks/scripts/run_accuracy.py @@ -26,11 +26,8 @@ import lm_eval import torch -UNIMODAL_MODEL_NAME = [ - "Qwen/Qwen2.5-7B-Instruct", "meta-llama/Llama-3.1-8B-Instruct", - "Qwen/Qwen3-8B" -] -UNIMODAL_TASK = ["ceval-valid", "mmlu", "gsm8k"] +UNIMODAL_MODEL_NAME = ["Qwen/Qwen2.5-7B-Instruct", "Qwen/Qwen3-8B"] +UNIMODAL_TASK = ["ceval-valid", "gsm8k"] MULTIMODAL_NAME = ["Qwen/Qwen2.5-VL-7B-Instruct"] MULTIMODAL_TASK = ["mmmu_val"] @@ -38,11 +35,6 @@ MODEL_RUN_INFO = { "Qwen/Qwen2.5-7B-Instruct": - ("export MODEL_ARGS='pretrained={model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" - "lm_eval --model vllm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" - "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" - ), - "meta-llama/Llama-3.1-8B-Instruct": ("export MODEL_ARGS='pretrained={model}, max_model_len=4096,dtype=auto,tensor_parallel_size=2,gpu_memory_utilization=0.6'\n" "lm_eval --model vllm --modlel_args $MODEL_ARGS --tasks {datasets} \ \n" "--apply_chat_template --fewshot_as_multiturn --num_fewshot 5 --batch_size 1" @@ -228,4 +220,7 @@ def main(args): parser.add_argument("--vllm_version", type=str, required=False) parser.add_argument("--cann_version", type=str, required=False) args = parser.parse_args() + # TODO(yikun): + # 1. add a exit 1 if accuracy is not as expected + # 2. Add ✅, ❌ to markdown if accuracy is not as expected main(args)