Fix bug in ChargeDistributionFilter for pedestal maker (cta-observatory#213)

tibaldo · jlenain · tibaldo · commit 2c24fafe7b93 · 2025-10-08T21:55:27.000+02:00
* Only fill charges container if waveforms are available

* Cleanup commented stuff from some previous PR

* Pin `persistent` (dependency for `ZODB`) version to fix CI

---------

Co-authored-by: Jean-Philippe Lenain &lt;jlenain@users.noreply.github.com&gt;
# Conflicts:
#	src/nectarchain/makers/component/pedestal_component.py
diff --git a/src/nectarchain/makers/component/pedestal_component.py b/src/nectarchain/makers/component/pedestal_component.py
@@ -306,7 +306,7 @@ def flag_bad_pixels(self, ped_stats, nevents):
         # Flag on standard deviation per pixel
         # Standard deviation of pedestal in channel/pixel above threshold
         log.info(
-            f"Flag pixels with pedestal standard deviation in a channel/pixel above "
+            f"Flag pixels with pedestal standard deviation in a chennel/pixel above "
             f"the maximum acceptable value {self.pixel_mask_std_pixel_max}"
         )
         flag_pixel_std = np.int8(

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ def flag_bad_pixels(self, ped_stats, nevents):`
`306`	`306`	`# Flag on standard deviation per pixel`
`307`	`307`	`# Standard deviation of pedestal in channel/pixel above threshold`
`308`	`308`	`log.info(`
`309`		`- f"Flag pixels with pedestal standard deviation in a channel/pixel above "`
	`309`	`+ f"Flag pixels with pedestal standard deviation in a chennel/pixel above "`
`310`	`310`	`f"the maximum acceptable value {self.pixel_mask_std_pixel_max}"`
`311`	`311`	`)`
`312`	`312`	`flag_pixel_std = np.int8(`