fix: reduction workflow for 3D BEER detectors

src/ess/beer/clustering.py

@@ -22,6 +22,9 @@ def cluster_events_by_streak(
     # adding them to the binned data coords achieves this.
     for coord in ('two_theta', 'Ltotal'):
         da.bins.coords[coord] = sc.bins_like(da, da.coords[coord])
+    # Making them scalar also keeps them after binning,
+    # but without having to store them for each event.
+    da.coords['tc'] = da.coords['tc'].mean()
 
     h = da.bins.concat().hist(coarse_d=1000)
     i_peaks, _ = find_peaks(

tests/diffraction/test_peaks.py

@@ -1,8 +1,23 @@
+import urllib.request
+
+import pytest
 import scipp as sc
 
 from ess.diffraction.peaks import dspacing_peaks_from_cif
 
 
+def _cifdb_is_reachable(timeout=5):
+    try:
+        req = urllib.request.Request('http://cod.ibt.lt', method="HEAD")
+        with urllib.request.urlopen(req, timeout=timeout):  # noqa: S310
+            return True
+    except Exception:
+        return False
+
+
+@pytest.mark.skipif(
+    not _cifdb_is_reachable(), reason='The CIF database can not be reached.'
+)
 def test_retreived_peak_list_has_expected_units():
     d = dspacing_peaks_from_cif(
         'codid::9011998',
@@ -12,6 +27,9 @@ def test_retreived_peak_list_has_expected_units():
     assert d.unit == 'barn'
 
 
+@pytest.mark.skipif(
+    not _cifdb_is_reachable(), reason='The CIF database can not be reached.'
+)
 def test_intensity_threshold_is_taken_into_account():
     d = dspacing_peaks_from_cif(
         'codid::9011998',
@@ -26,6 +44,9 @@ def test_intensity_threshold_is_taken_into_account():
     assert len(d) == 0
 
 
+@pytest.mark.skipif(
+    not _cifdb_is_reachable(), reason='The CIF database can not be reached.'
+)
 def test_retreived_peak_list_with_temperature_kwarg():
     d = dspacing_peaks_from_cif(
         'codid::9011998', sc.scalar(50, unit='barn'), uiso_temperature=300