Skip to content

Commit 0cf4917

Browse files
authored
Remove latest v4 test to reduce cost (#1098)
## Summary <!--- This is a required section; please describe the main purpose of this proposed code change. ---> We have too many tests right now. It costs too much. <!--- ## Details This is an optional section; is there anything specific that reviewers should be aware of? ---> ## Testing Done <!--- This is a required section; please describe how this change was tested. ---> <!-- Replace BLANK with your device type. For example, A100-80G-PCIe Complete the following tasks before sending your PR, and replace `[ ]` with `[x]` to indicate you have done them. --> - Hardware Type: <BLANK> - [ ] run `make test` to ensure correctness - [x] run `make checkstyle` to ensure code style - [ ] run `make test-convergence` to ensure convergence
1 parent 175cdf0 commit 0cf4917

File tree

2 files changed

+0
-90
lines changed

2 files changed

+0
-90
lines changed

.github/workflows/nvi-ci.yml

Lines changed: 0 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -148,55 +148,3 @@ jobs:
148148
- name: Run tests
149149
run: |
150150
modal run dev.modal.tests::liger_oldest_v4_convergence_tests
151-
152-
correctness-tests-with-transformers-4-57-6:
153-
runs-on: ubuntu-latest
154-
needs: [checkstyle]
155-
env:
156-
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }}
157-
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }}
158-
159-
steps:
160-
- name: Checkout code
161-
uses: actions/checkout@v6
162-
163-
- name: Set up Python
164-
uses: actions/setup-python@v6
165-
with:
166-
python-version: '3.10'
167-
168-
- name: Install dependencies
169-
run: |
170-
python -m pip install --upgrade pip
171-
pip install modal
172-
173-
- name: Run tests
174-
run: |
175-
modal run dev.modal.tests::liger_latest_v4_correctness_tests
176-
177-
178-
179-
convergence-tests-with-transformers-4-57-6:
180-
runs-on: ubuntu-latest
181-
needs: [checkstyle]
182-
env:
183-
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }}
184-
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }}
185-
186-
steps:
187-
- name: Checkout code
188-
uses: actions/checkout@v6
189-
190-
- name: Set up Python
191-
uses: actions/setup-python@v6
192-
with:
193-
python-version: '3.10'
194-
195-
- name: Install dependencies
196-
run: |
197-
python -m pip install --upgrade pip
198-
pip install modal
199-
200-
- name: Run tests
201-
run: |
202-
modal run dev.modal.tests::liger_latest_v4_convergence_tests

dev/modal/tests.py

Lines changed: 0 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -84,41 +84,3 @@ def liger_oldest_v4_convergence_tests():
8484

8585

8686
latest_v4_app = modal.App("liger_latest_v4_tests", image=image) # 4.57.6
87-
88-
89-
@latest_v4_app.function(gpu="H100!", image=repo, timeout=90 * 60)
90-
def liger_latest_v4_correctness_tests():
91-
import subprocess
92-
93-
subprocess.run(
94-
["uv pip install -e '.[dev]' --system"],
95-
check=True,
96-
shell=True,
97-
cwd=REMOTE_ROOT_PATH,
98-
)
99-
subprocess.run(
100-
[f"uv pip install 'transformers>={OLDEST_SUPPORTED_TRANSFORMERS_V4_VERSION}, <5.0.0' --system"],
101-
check=True,
102-
shell=True,
103-
cwd=REMOTE_ROOT_PATH,
104-
)
105-
subprocess.run(["make test"], check=True, shell=True, cwd=REMOTE_ROOT_PATH)
106-
107-
108-
@latest_v4_app.function(gpu="H100!", image=repo, timeout=90 * 60)
109-
def liger_latest_v4_convergence_tests():
110-
import subprocess
111-
112-
subprocess.run(
113-
["uv pip install -e '.[dev]' --system"],
114-
check=True,
115-
shell=True,
116-
cwd=REMOTE_ROOT_PATH,
117-
)
118-
subprocess.run(
119-
[f"uv pip install 'transformers>={OLDEST_SUPPORTED_TRANSFORMERS_V4_VERSION}, <5.0.0' --system"],
120-
check=True,
121-
shell=True,
122-
cwd=REMOTE_ROOT_PATH,
123-
)
124-
subprocess.run(["make test-convergence"], check=True, shell=True, cwd=REMOTE_ROOT_PATH)

0 commit comments

Comments
 (0)