Remove Datablock (#570)

Datablock has been deprecated for several years now. The dxtbx.datablock
module has been left in as an alias to the functionality now moved to
dxtbx.experiment_list, but will raise a deprecation warning pointing to
the new place.

Co-authored-by: Aaron Brewster <asbrewster@lbl.gov>
Co-authored-by: David Waterman <dgwaterman@gmail.com>
Co-authored-by: Nicholas Devenish <ndevenish@gmail.com>

newsfragments/504.removal

@@ -0,0 +1 @@
+dxtbx: remove legacy datablock object (obsolete for several years)

pytest.ini

@@ -3,7 +3,6 @@ addopts = -rsxX
 filterwarnings =
     ignore:the matrix subclass is not the recommended way:PendingDeprecationWarning
     ignore:numpy.dtype size changed:RuntimeWarning
-    ignore:Datablocks are deprecated:UserWarning
     ignore:Deprecated call to `pkg_resources.declare_namespace:DeprecationWarning
     ignore:`product` is deprecated as of NumPy:DeprecationWarning:h5py|numpy
 junit_family = legacy

src/dxtbx/command_line/display_parallax_correction.py

@@ -7,15 +7,17 @@
 from scitbx.array_family import flex
 
 import dxtbx.util
-from dxtbx.datablock import DataBlockFactory
 from dxtbx.model import ParallaxCorrectedPxMmStrategy
+from dxtbx.model.experiment_list import ExperimentListFactory
 
 
 def run(args=None):
     dxtbx.util.encode_output_as_utf8()
-    datablocks = DataBlockFactory.from_args(args or sys.argv[1:])
-    assert len(datablocks) == 1
-    detectors = datablocks[0].unique_detectors()
+    experiments = ExperimentListFactory.from_args(
+        args or sys.argv[1:], check_format=False
+    )
+    assert len(experiments) == 1
+    detectors = experiments.detectors()
     assert len(detectors) == 1
     detector = detectors[0]
     assert len(detector) == 1

src/dxtbx/command_line/plot_detector_models.py

@@ -17,12 +17,11 @@
 from scitbx.matrix import col
 
 import dxtbx.util
-from dxtbx.datablock import DataBlockFactory
 from dxtbx.model.detector_helpers import get_detector_projection_2d_axes
 from dxtbx.model.experiment_list import ExperimentListFactory
 
 usage = """Plot dxtbx detector models. Provide multiple json files if desired
-Example: dxtbx.plot_detector_models datablock1.json datablock2.json
+Example: dxtbx.plot_detector_models experiments1.json experiments2.json
 """
 
 
@@ -181,16 +180,12 @@ def run(args=None):
 
         # read the data and get the detector models
         try:
-            datablocks = DataBlockFactory.from_json_file(file_name, check_format=False)
-            detectors = sum((db.unique_detectors() for db in datablocks), [])
-        except Exception:
-            try:
-                experiments = ExperimentListFactory.from_json_file(
-                    file_name, check_format=False
-                )
-            except ValueError:
-                experiments = ExperimentListFactory.from_filenames([file_name])
-            detectors = experiments.detectors()
+            experiments = ExperimentListFactory.from_json_file(
+                file_name, check_format=False
+            )
+        except ValueError:
+            experiments = ExperimentListFactory.from_filenames([file_name])
+        detectors = experiments.detectors()
         if not params.plot_all_detectors:
             detectors = detectors[0:1]
         for detector in detectors: