Handle missing Hlt2 SelReports in Tesla

Occasionally, HLT2 did not write Hlt2/SelReports for very busy events. Previously, the TeslaReportAlgo was skipping such events happily.

Here we restore this behaviour, such that if Hlt2/SelReports cannot be decoded, no event is written out.

This was spotted in the validation productions, see https://lblogbook.cern.ch/Operations/27130

changed the description

cc @apearce

Looks good to me. Long term we might want the option to switch that check off, it did help to spot a problem.

ideally, we shouldn't use the sel reports for the RecSummary.

• [2017-07-11 19:30] Validation started with lhcb-gaudi-merge#167
• [2017-07-12 00:05] Validation started with lhcb-old-stripping-tests#462
• [2017-07-12 00:09] Validation started with lhcb-2017-patches#76
• [2017-07-12 00:10] Validation started with lhcb-head#1533
• [2017-07-12 00:10] Validation started with lhcb-gaudi-head-noavx2#48
• [2017-07-12 00:10] Validation started with lhcb-clang-test#640
• [2017-07-12 00:10] Validation started with lhcb-lcg-dev4#217
• [2017-07-12 00:10] Validation started with lhcb-lcg-dev3#217
• [2017-07-12 00:10] Validation started with lhcb-gaudi-head-noavx#67
• [2017-07-12 00:11] Validation started with lhcb-gaudi-head#1530
• [2017-07-12 08:10] Validation started with lhcb-gaudi-merge#168
• [2017-07-12 18:22] Validation started with lhcb-gaudi-head#720
• [2017-07-13 00:05] Validation started with lhcb-old-stripping-tests#463
• [2017-07-13 00:08] Validation started with lhcb-2017-patches#77
• [2017-07-13 00:11] Validation started with lhcb-lcg-dev4#218
• [2017-07-13 00:12] Validation started with lhcb-head#1534
• [2017-07-13 00:12] Validation started with lhcb-gaudi-head-noavx2#49
• [2017-07-13 00:12] Validation started with lhcb-clang-test#641
• [2017-07-13 00:13] Validation started with lhcb-gaudi-head#1531
• [2017-07-13 00:30] Validation started with lhcb-lcg-dev3#218
• [2017-07-13 00:30] Validation started with lhcb-gaudi-head-noavx#68
• [2017-07-13 08:10] Validation started with lhcb-gaudi-merge#169
• [2017-07-13 08:10] Validation started with lhcb-gaudi-merge#170
• [2017-07-14 00:04] Validation started with lhcb-old-stripping-tests#464
• [2017-07-14 00:09] Validation started with lhcb-2017-patches#78
• [2017-07-14 00:11] Validation started with lhcb-gaudi-head#1532
• [2017-07-14 00:11] Validation started with lhcb-lcg-dev3#219
• [2017-07-14 00:11] Validation started with lhcb-clang-test#642
• [2017-07-14 00:12] Validation started with lhcb-gaudi-head-noavx2#50
• [2017-07-14 00:12] Validation started with lhcb-head#1535
• [2017-07-14 00:12] Validation started with lhcb-lcg-dev4#219
• [2017-07-15 00:05] Validation started with lhcb-old-stripping-tests#465
• [2017-07-15 00:08] Validation started with lhcb-2017-patches#79
• [2017-07-15 00:09] Validation started with lhcb-head#1536
• [2017-07-15 00:10] Validation started with lhcb-clang-test#643
• [2017-07-15 00:10] Validation started with lhcb-lcg-dev3#220
• [2017-07-15 00:10] Validation started with lhcb-gaudi-head-noavx#70
• [2017-07-15 00:10] Validation started with lhcb-lcg-dev4#220
• [2017-07-15 00:10] Validation started with lhcb-gaudi-head-noavx2#51
• [2017-07-15 00:12] Validation started with lhcb-gaudi-head#1533
• [2017-07-16 00:04] Validation started with lhcb-old-stripping-tests#466
• [2017-07-16 00:09] Validation started with lhcb-2017-patches#80
• [2017-07-16 00:09] Validation started with lhcb-head#1537
• [2017-07-16 00:10] Validation started with lhcb-gaudi-head-noavx#71
• [2017-07-16 00:10] Validation started with lhcb-lcg-dev3#221
• [2017-07-16 00:10] Validation started with lhcb-gaudi-head-noavx2#52
• [2017-07-16 00:10] Validation started with lhcb-clang-test#644
• [2017-07-16 00:11] Validation started with lhcb-lcg-dev4#221
• [2017-07-16 00:11] Validation started with lhcb-gaudi-head#1534
• [2017-07-17 00:04] Validation started with lhcb-old-stripping-tests#467
• [2017-07-17 00:08] Validation started with lhcb-2017-patches#81
• [2017-07-17 00:10] Validation started with lhcb-gaudi-head-noavx#72
• [2017-07-17 00:10] Validation started with lhcb-head#1538
• [2017-07-17 00:11] Validation started with lhcb-clang-test#645
• [2017-07-17 00:11] Validation started with lhcb-lcg-dev4#222
• [2017-07-17 00:11] Validation started with lhcb-gaudi-head-noavx2#53
• [2017-07-17 00:12] Validation started with lhcb-gaudi-head#1535

As the location is parameterised, the configurable name should be as well.

def _selReportsCheck(self, loc='Hlt2/SelReports'):
    name = '{0}Check'.format(loc.replace('/', ''))
    return TESCheck(name, Inputs=[loc], Stop=False)

LGTM thanks a lot

merged

mentioned in commit f0c00377

mentioned in commit ee02442c

mentioned in commit bb49cf21

mentioned in merge request !80 (merged)

mentioned in commit 69335459