Skip to content

Commit

Permalink
Support github matrix workflow for many rays (#148)
Browse files Browse the repository at this point in the history
---------

Signed-off-by: Qing Wang <kingchin1218@gmail.com>
  • Loading branch information
jovany-wang committed Jul 11, 2023
1 parent a0bba61 commit 0ce3f60
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 45 deletions.
38 changes: 0 additions & 38 deletions .github/workflows/test_on_ray2.4.0.yml

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: test on ray2.0.0
name: test on many rays

on:
push:
Expand All @@ -7,7 +7,12 @@ on:
branches: [ main ]

jobs:
run-unit-tests:
run-unit-tests-on-many-rays:
strategy:
matrix:
os: [ubuntu-latest]
ray_version: [2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.4.0]

timeout-minutes: 60
runs-on: ubuntu-latest
container: docker.io/library/ubuntu:latest
Expand All @@ -28,7 +33,9 @@ jobs:
. py3/bin/activate
which python
pip install pytest torch cloudpickle cryptography
pip install ray==2.0.0
# six is required in ray-2.1.0
pip install "protobuf<4.0" six
pip install ray==${{ matrix.ray_version }}
- name: Build and test
run: |
Expand Down
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,10 @@ The code view in each party is exactly the same, but the execution differs based


## Supported Ray Versions
| RayFed Versions | ray-1.13.0 | ray-2.0.0 | ray-2.1.0 | ray-2.2.0 | ray-2.3.0 |
|:---------------:|:--------:|:--------:|:--------:|:--------:|:--------:|
| 0.1.0 ||||| |
| 0.2.0 |not released|not released|not released|not released|not released|
| RayFed Versions | ray-1.13.0 | ray-2.0.0 | ray-2.1.0 | ray-2.2.0 | ray-2.3.0 | ray-2.4.0 |
|:---------------:|:--------:|:--------:|:--------:|:--------:|:--------:|:--------:|
| 0.1.0 ||||| | |
| 0.2.0 |not released|not released|not released|not released|not released|not released|


## Installation
Expand Down

0 comments on commit 0ce3f60

Please sign in to comment.