Skip to content

Commit

Permalink
Merge pull request #101 from lsst-camera-dh/LSSTTD-1554_skip_missing_…
Browse files Browse the repository at this point in the history
…rafts_for_signal_correlation_task

skip signal correlation calculation for missing rafts and report the …
  • Loading branch information
jchiang87 authored Feb 7, 2021
2 parents 44dbdcb + 807f5bb commit 3c94071
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,10 @@ def raft_jh_signal_correlations(raft_name):
folders = sorted(list(set([os.path.basename(os.path.dirname(_))
for _ in sflat_files])))
logger.info(f'folders: {folders}')
if not folders:
logger.info('No data found for this raft, so skip the '
'signal correlation analysis.')
return

flat1_files = dict()
flat2_files = dict()
Expand Down
7 changes: 7 additions & 0 deletions python/bot_eo_validators.py
Original file line number Diff line number Diff line change
Expand Up @@ -534,14 +534,21 @@ def validate_flat_pairs(results, det_names):
metadata=metadata))

# Persist the raft-level imaging region correlation plots.
missing_raft_names = set()
for raft in camera_info.get_installed_raft_names():
metadata = dict(TESTTYPE='FLAT', TEST_CATEGORY='EO', RAFT=raft, RUN=run)
file_prefix = make_file_prefix(run, raft)
filename = f'{file_prefix}_imaging_region_correlations.png'
if not os.path.isfile(filename):
missing_raft_names.add(raft)
continue
results.extend(siteUtils.persist_png_files(filename, file_prefix,
metadata=metadata))

report_missing_data("validate_flat_pairs", missing_det_names)
report_missing_data("validate_flat_pairs",
sorted(list(missing_raft_names)),
components='rafts', total=21)

return results

Expand Down

0 comments on commit 3c94071

Please sign in to comment.