Skip to content

Commit 456e102

Browse files
author
Mark Saroufim
authored
Revert "Use Modal for CI" (#314)
* Revert "Use Modal for CI (#311)" This reverts commit e413950. * Trigger CI * update
1 parent 706a908 commit 456e102

6 files changed

Lines changed: 52 additions & 196 deletions

File tree

.github/workflows/runner_ci.yml

Lines changed: 47 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,25 +11,59 @@ on:
1111
- dev
1212

1313
jobs:
14-
check-modal:
15-
runs-on: ubuntu-latest
16-
timeout-minutes: 30
14+
check-cuda:
15+
runs-on: [gpumode-nvidia-arc]
16+
timeout-minutes: 10
17+
container:
18+
image: nvidia/cuda:12.4.0-devel-ubuntu22.04
1719
steps:
1820
- uses: actions/checkout@v3
1921

2022
- name: Setup Python
2123
uses: actions/setup-python@v5
2224
with:
23-
python-version: '3.13'
25+
python-version: '3.10'
2426

25-
- name: Install dependencies
26-
run: |
27-
pip install modal
28-
pip install -r requirements.txt
27+
- name: Install pytest
28+
shell: bash
29+
run: pip install pytest
30+
31+
- name: Run script
32+
shell: bash
33+
run: pytest scripts/ci_test_cuda.py
34+
35+
env:
36+
CUDA_VISIBLE_DEVICES: 0
37+
38+
check-pytorch:
39+
runs-on: [gpumode-nvidia-arc]
40+
timeout-minutes: 10
41+
container:
42+
image: nvidia/cuda:12.4.0-devel-ubuntu22.04
43+
steps:
44+
- uses: actions/checkout@v3
2945

30-
- name: Run Modal tests
46+
- name: Setup Python
47+
uses: actions/setup-python@v5
48+
with:
49+
python-version: '3.10'
50+
51+
- name: Install uv
52+
uses: astral-sh/setup-uv@v3
53+
with:
54+
version: "latest"
55+
56+
- name: Setup Python environment
3157
run: |
32-
python scripts/modal_ci_test.py
33-
env:
34-
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }}
35-
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }}
58+
uv venv .venv
59+
echo "VIRTUAL_ENV=$PWD/.venv" >> $GITHUB_ENV
60+
echo "$PWD/.venv/bin" >> $GITHUB_PATH
61+
uv pip install numpy torch setuptools ninja pytest
62+
63+
- name: Run script
64+
shell: bash
65+
run: pytest scripts/ci_test_python.py
66+
67+
env:
68+
CUDA_VISIBLE_DEVICES: 0
69+

requirements.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ yoyo-migrations
1010
ruff
1111
pre-commit
1212
better_profanity
13-
pytest
1413

1514
# api
1615
fastapi[all] # install all to avoid random bugs

scripts/ci_test_python.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def custom_kernel(input):
5959
assert run.result["test.0.status"] == "fail"
6060
assert (
6161
run.result["test.0.error"]
62-
== "mismatch found! custom implementation doesn't match reference.:"
62+
== "mismatch found! custom implementation doesn't match reference:"
6363
" Number of mismatched elements: 256"
6464
)
6565
assert run.exit_code == ExitCode.VALIDATE_FAIL

scripts/modal_ci_test.py

Lines changed: 0 additions & 174 deletions
This file was deleted.

src/discord-cluster-manager/modal_runner.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,18 +28,16 @@
2828
"requests~=2.32.4",
2929
"packaging~=25.0",
3030
"numpy~=2.3",
31-
"pytest",
32-
3331
)
3432
.pip_install(
3533
"torch~=2.7",
3634
"torchvision~=0.22",
3735
"torchaudio~=2.7",
38-
index_url="https://download.pytorch.org/whl/cu128",
36+
index_url="https://download.pytorch.org/whl/cu128"
3937
)
4038
# other frameworks
4139
.pip_install(
42-
"jax[cuda12]==0.5.3", # 0.6 want's cudnn 9.8 in conflict with torch 2.7
40+
"jax[cuda12]==0.5.3", # 0.6 want's cudnn 9.8 in conflict with torch 2.7
4341
"jax2torch==0.0.7",
4442
"tinygrad~=0.10",
4543
)
@@ -49,8 +47,8 @@
4947
"nvidia-cutlass-dsl~=4.0",
5048
"cuda-core[cu12]~=0.3",
5149
"cuda-python[all]==12.8",
52-
# "nvmath-python[cu12]~=0.4",
53-
# "numba-cuda[cu12]~=0.15",
50+
#"nvmath-python[cu12]~=0.4",
51+
#"numba-cuda[cu12]~=0.15",
5452
)
5553
)
5654

src/discord-cluster-manager/task.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,5 +191,4 @@ def build_task_config(
191191
"sources": sources,
192192
"headers": headers,
193193
"include_dirs": task.config.include_dirs,
194-
**common,
195194
}

0 commit comments

Comments
 (0)