Skip to content

Commit fb90a35

Browse files
Remove test_io_set_preload_false_is_faster (#13933)
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 6b7c6f4 commit fb90a35

1 file changed

Lines changed: 0 additions & 21 deletions

File tree

mne/io/eeglab/tests/test_eeglab.py

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
import os
66
import shutil
7-
import time
87
from copy import deepcopy
98

109
import numpy as np
@@ -768,26 +767,6 @@ def test_eeglab_drop_nan_annotations(tmp_path):
768767
raw = read_raw_eeglab(file_path, preload=True)
769768

770769

771-
@pytest.mark.flaky
772-
@testing.requires_testing_data
773-
@pytest.mark.timeout(10)
774-
@pytest.mark.slowtest # has the advantage of not running on macOS where it errs a lot
775-
def test_io_set_preload_false_is_faster():
776-
"""Using preload=False should skip the expensive data read branch."""
777-
# warm start
778-
read_raw_eeglab(raw_fname_mat, preload=False)
779-
780-
durations = {}
781-
for preload in (True, False):
782-
start = time.perf_counter()
783-
_ = read_raw_eeglab(raw_fname_mat, preload=preload)
784-
durations[preload] = time.perf_counter() - start
785-
786-
# preload=True should not be faster than preload=False (timings may vary
787-
# across systems, so avoid strict thresholds)
788-
assert durations[True] > durations[False]
789-
790-
791770
@testing.requires_testing_data
792771
def test_lazy_vs_preload_integrity():
793772
"""Test that lazy loading produces identical data to preload."""

0 commit comments

Comments
 (0)