Merge pull request #2680 from cta-observatory/hdf5_true_image_sum

Fill `containers.SimulatedCameraContainer.true_image_sum` in HDF5eventsource
cta-observatory · Jan 21, 2025 · 1644e85 · 1644e85
2 parents 708dbfc + bed2598
commit 1644e85
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -1,6 +1,12 @@
 ctapipe v0.23.2 (2025-01-21)
 ============================
 
+Bug Fixes
+---------
+
+- Fill ``ctapipe.containers.SimulatedCameraContainer.true_image_sum`` in
+  ``HDF5EventSource``. Always returned default value of -1 before the fix. [`#2680 <https://github.com/cta-observatory/ctapipe/pull/2680>`__]
+
 Maintenance
 -----------
 

diff --git a/src/ctapipe/io/hdf5eventsource.py b/src/ctapipe/io/hdf5eventsource.py
@@ -708,6 +708,7 @@ def _generator(self):
                     if self.has_simulated_dl1:
                         simulated_image_row = next(simulated_image_iterators[key])
                         simulated.true_image = simulated_image_row["true_image"]
+                        simulated.true_image_sum = simulated_image_row["true_image_sum"]
 
                 if DataLevel.DL1_PARAMETERS in self.datalevels:
                     # Is there a smarter way to unpack this?