Skip to content

[LPD][DARK] skip characterize_module and skip plots for module with no data

Karim Ahmed requested to merge fix/empty_cellid into master

A fix for LPD darks for a bug caught while testing 3.8.0

Description

http://max-exfl016.desy.de:8008/gpfs/exfel/d/cal/caldb_store/xfel/reports/CALLAB/FXE_DET_LPD1M-1/dark/dark_900203_r9008_r9009_r9010_230116_161256.pdf

For modules with no data, an error is raised:

"""
Traceback (most recent call last):
File "/home/xcal/.pyenv/versions/3.8.11/lib/python3.8/multiprocessing/pool.py",
˓→line 125, in worker
result = (True, func(*args, **kwds))
File "/home/xcal/.pyenv/versions/3.8.11/lib/python3.8/multiprocessing/pool.py",
˓→line 51, in starmapstar
return list(itertools.starmap(args[0], args[1]))
File "<ipython-input-5-fad35870868c>", line 31, in characterize_module
cellid_pattern = cellid[:count[0]]
IndexError: index 0 is out of bounds for axis 0 with size 0
"""

I tried to fix it in the shortest way by skipping characterize_module and skipping plots for this module.

This fix is live in production

http://max-exfl016.desy.de:8008/gpfs/exfel/d/cal/caldb_store/xfel/reports/CALLAB/FXE_DET_LPD1M-1/dark/dark_900203_r9008_r9009_r9010_230116_200315.pdf

How Has This Been Tested?

Tested locally and on production

Relevant Documents (optional)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

Reviewers

@schmidtp @kluyvert

Edited by Karim Ahmed

Merge request reports