Fix export to EDF format with a set physical range smaller than the data range (#11569)

Author: Mathieu Scheltienne

doc/changes/latest.inc

@@ -56,6 +56,7 @@ Bugs
 - Expand tilde (user directory) in config keys (:gh:`11537` by `Clemens Brunner`_)
 - Fix bug in :func:`mne.preprocessing.compute_maxwell_basis` where using ``int_order=0`` would raise an error (:gh:`11562` by `Eric Larson`_)
 - Fix :func:`mne.io.read_raw` for file names containing multiple dots (:gh:`11521` by `Clemens Brunner`_)
+- Fix bug in :func:`mne.export.export_raw` when exporting to EDF with a physical range set smaller than the data range (:gh:`11569` by `Mathieu Scheltienne`_)
 
 API changes
 ~~~~~~~~~~~

mne/export/_edf.py

@@ -142,11 +142,15 @@ def _export_raw(fname, raw, physical_range, add_ch_type):
 
         # check that physical min and max is not exceeded
         if data.max() > pmax:
-            raise RuntimeError(f'The maximum μV of the data {data.max()} is '
-                               f'more than the physical max passed in {pmax}.')
+            warn(
+                f"The maximum μV of the data {data.max()} is "
+                f"more than the physical max passed in {pmax}.",
+            )
         if data.min() < pmin:
-            raise RuntimeError(f'The minimum μV of the data {data.min()} is '
-                               f'less than the physical min passed in {pmin}.')
+            warn(
+                f"The minimum μV of the data {data.min()} is "
+                f"less than the physical min passed in {pmin}.",
+            )
 
     # create instance of EDF Writer
     with _auto_close(EDFwriter(fname, file_type, n_channels)) as hdl:

mne/export/tests/test_export.py

@@ -6,8 +6,7 @@
 
 from contextlib import nullcontext
 from datetime import datetime, timezone
-from mne.io import RawArray
-from mne.io.meas_info import create_info
+from os import remove
 from pathlib import Path
 
 import pytest
@@ -20,8 +19,9 @@
 from mne.datasets import testing, misc
 from mne.export import export_evokeds, export_evokeds_mff
 from mne.fixes import _compare_version
-from mne.io import (read_raw_fif, read_raw_eeglab, read_raw_edf,
+from mne.io import (RawArray, read_raw_fif, read_raw_eeglab, read_raw_edf,
                     read_raw_brainvision)
+from mne.io.meas_info import create_info
 from mne.utils import (_check_eeglabio_installed, requires_version,
                        object_diff, _check_edflib_installed, _resource_path,
                        _check_pybv_installed, _record_warnings)
@@ -309,12 +309,23 @@ def test_export_raw_edf(tmp_path, dataset, format):
     temp_fname = tmp_path / f"test.{format}"
 
     # test runtime errors
-    with pytest.raises(RuntimeError, match='The maximum'), \
-            pytest.warns(RuntimeWarning, match='Data has a non-integer'):
+    with pytest.warns() as record:
         raw.export(temp_fname, physical_range=(-1e6, 0))
-    with pytest.raises(RuntimeError, match='The minimum'), \
-            pytest.warns(RuntimeWarning, match='Data has a non-integer'):
+    if dataset == 'test':
+        assert any(
+            "Data has a non-integer" in str(rec.message) for rec in record
+        )
+    assert any("The maximum" in str(rec.message) for rec in record)
+    remove(temp_fname)
+
+    with pytest.warns() as record:
         raw.export(temp_fname, physical_range=(0, 1e6))
+    if dataset == 'test':
+        assert any(
+            "Data has a non-integer" in str(rec.message) for rec in record
+        )
+    assert any("The minimum" in str(rec.message) for rec in record)
+    remove(temp_fname)
 
     if dataset == 'test':
         with pytest.warns(RuntimeWarning, match='Data has a non-integer'):