[reward] refactor: remove reward model worker code and invocation #1403
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| # # Tests layout | |
| # Each folder under tests/ corresponds to a test category for a sub-namespace in verl. For instance: | |
| # - `tests/trainer` for testing functionality related to `verl/trainer` | |
| # - `tests/models` for testing functionality related to `verl/models` | |
| # - ... | |
| # There are a few folders with `special_` prefix, created for special purposes: | |
| # - `special_distributed`: unit tests that must run with multiple GPUs | |
| # - `special_e2e`: end-to-end tests with training/generation scripts | |
| # - `special_npu`: tests for NPUs | |
| # - `special_sanity`: a suite of quick sanity tests | |
| # - `special_standalone`: a set of test that are designed to run in dedicated environments | |
| # Accelerators for tests | |
| # - By default tests are run with GPU available, except for the ones under `special_npu`, and any test script whose name ends with `on_cpu.py`. | |
| # - For test scripts with `on_cpu.py` name suffix would be tested on CPU resources in linux environment. | |
| # # Workflow layout | |
| # All CI tests are configured by yaml files in `.github/workflows/`. Here's an overview of all test configs: | |
| # 1. A list of always triggered CPU sanity tests: `check-pr-title.yml`, `secrets_scan.yml`, `check-pr-title,yml`, `pre-commit.yml`, `doc.yml` | |
| # 2. Some heavy multi-GPU unit tests, such as `model.yml`, `vllm.yml`, `sgl.yml` | |
| # 3. End-to-end tests: `e2e_*.yml` | |
| # 4. Unit tests | |
| # - `cpu_unit_tests.yml`, run pytest on all scripts with file name pattern `tests/**/test_*_on_cpu.py` | |
| # - `gpu_unit_tests.yml`, run pytest on all scripts with file without the `on_cpu.py` suffix. | |
| # - `npu_unit_tests.yml`, run pytest on all scripts with file without the `on_cpu.py` suffix on ascend device. | |
| # - Since cpu/gpu/npu unit tests by default runs all tests under `tests`, please make sure tests are manually excluded in them when | |
| # - new workflow yaml is added to `.github/workflows` | |
| # - new tests are added to workflow mentioned in 2. | |
| name: NPU unit tests | |
| on: | |
| # Trigger the workflow on push or pull request, | |
| # but only for the main branch | |
| push: | |
| branches: | |
| - main | |
| - v0.* | |
| paths: | |
| - "**/*.py" | |
| - .github/workflows/npu_unit_tests.yml | |
| pull_request: | |
| branches: | |
| - main | |
| paths: | |
| # The order that you define paths patterns matters: | |
| # A matching negative pattern (prefixed with !) after a positive match will exclude the path. | |
| # A matching positive pattern after a negative match will include the path again. | |
| - "**/*.py" | |
| # Other entrypoints | |
| - "!examples/**" | |
| - "!verl/trainer/main_*.py" | |
| - "!verl/trainer/fsdp_sft_trainer.py" | |
| - "!recipe/**" | |
| # Entrypoints | |
| - .github/workflows/npu_unit_tests.yml | |
| - "tests/**test_*.py" | |
| # Ignore CPU tests | |
| - "!tests/*_on_cpu.py" | |
| # Cancel jobs on the same ref if a new one is triggered | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.ref }} | |
| cancel-in-progress: ${{ github.ref != 'refs/heads/main' }} | |
| # Declare permissions just read content. | |
| permissions: | |
| contents: read | |
| jobs: | |
| npu_unit_tests: | |
| if: github.repository_owner == 'verl-project' | |
| runs-on: linux-aarch64-a2-8 | |
| timeout-minutes: 60 # Increase this timeout value as needed | |
| container: | |
| image: swr.ap-southeast-1.myhuaweicloud.com/base_image/ascend-ci/verl/verl:verl-8.3.rc1-910b-ubuntu22.04-py3.11-latest | |
| options: >- | |
| --shm-size 16g | |
| env: | |
| HF_ENDPOINT: "https://hf-mirror.com" | |
| HF_HUB_ENABLE_HF_TRANSFER: "0" # This is more stable | |
| steps: | |
| - name: Check npu and CANN info | |
| run: | | |
| cat /usr/local/Ascend/ascend-toolkit/latest/"$(uname -i)"-linux/ascend_toolkit_install.info | |
| npu-smi info | |
| - name: Check initial pip list from image | |
| run: | | |
| pip list | |
| - name: Checkout volcengine/verl repo | |
| uses: actions/checkout@v4 | |
| with: | |
| fetch-depth: 0 | |
| clean: true | |
| - name: Install the current repository | |
| run: | | |
| pip install -r requirements-npu.txt | |
| pip install --no-deps -e .[test] | |
| pip install mlflow pytest-asyncio | |
| - name: Check final pip list | |
| run: | | |
| pip list | |
| - name: Prepare weights | |
| run: | | |
| ln -s /root/.cache/models ~/models | |
| - name: Run all NPU unit tests | |
| run: | | |
| export PYTHONPATH=$PYTHONPATH:/Megatron-LM | |
| pytest -s -x --ignore-glob="*test_special_*.py" --ignore-glob="*on_cpu.py" --ignore-glob="*test_vllm*" --ignore-glob="*_sglang*" --ignore-glob="*_hf_rollout*" --ignore-glob="tests/models/" --ignore-glob="tests/special*" --ignore-glob="tests/experimental" --ignore-glob="tests/workers/reward_model" --ignore-glob="*test_rvdz*" --ignore-glob="*test_ray_collectives*" --ignore-glob="*test_nvtx_profile*" --ignore-glob="tests/checkpoint_engine" --ignore-glob="*test_shared_memory*" --ignore-glob="tests/workers/rollout/rollout_trtllm" tests/ | |
| - name: Testing FSDP2 actor functionality | |
| run: | | |
| torchrun --standalone --nnodes=1 --nproc-per-node=2 tests/workers/actor/test_special_dp_actor.py | |
| - name: Testing FSDP2 critic functionality | |
| run: | | |
| torchrun --standalone --nnodes=1 --nproc-per-node=2 tests/workers/critic/test_special_dp_critic.py | |
| - name: Running NPU profiling unit tests | |
| run: | | |
| pytest -s -x tests/utils/test_special_mstx_profile.py |