Skip to content

Commit e38f5da

Browse files
committed
fix(dpa-adapt): drop dead setup in skipped CV test
test_deterministic_folds_same_result_twice unconditionally skips (needs a real DPA checkpoint), but still ran stub setup code whose `systems` local was flagged by CodeQL as unused. Reduce it to a bare skip, matching the sibling test_manifest_folds.
1 parent 5e082a0 commit e38f5da

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

source/tests/dpa_adapt/test_split_cv.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -255,12 +255,6 @@ class TestDeterministicCV:
255255
"""Ensures cross_validate with frozen_sklearn + GroupKFold is deterministic."""
256256

257257
def test_deterministic_folds_same_result_twice(self, tmp_path, monkeypatch):
258-
formulas = [f"Comp{i}" for i in range(4)]
259-
systems = _write_oer_tree(str(tmp_path), formulas, nsets=2, label_key="energy")
260-
261-
rng = np.random.default_rng(42)
262-
n_total = len(systems) * 3 # 3 frames each
263-
n_total = sum(1 for _ in tmp_path.rglob("set.000"))
264258
raise pytest.skip("needs real DPA checkpoint to extract descriptors")
265259

266260
def test_manifest_folds(self, tmp_path, monkeypatch):

0 commit comments

Comments
 (0)