Modified CbmBuildEventsQa and QA function in CbmSeedFinderSlidingWindow to...

Modified CbmBuildEventsQa and QA function in CbmSeedFinderSlidingWindow to treat digis with empty matches as noise.

Supercedes !686 (closed)

This is much safer than simply ignoring the empty digi matches.

mentioned in merge request !689 (merged)

mentioned in merge request !686 (closed)

Additional remark: The issue raised by S. Roy in the Software meeting, whereby every trigger was matched to MC event -1, was also caused by the presence of T0 digis, and illustrates the danger of using TOF as a trigger detector.

The issue was caused by setting TOF as the trigger detector, setting a digi count threshold of "1" and a large dead time. The trigger algorithm then triggered for each T0 digi, and discarded all remaining digis in the time slice. The trigger algorithm QA then matched triggers with no links to MC event -1 (in that specific circumstance empty matches were already treated the same as noise).

Until a selection procedure is implemented, which rejects T0 digis based on their address, using TOF as a trigger detector is probably not advisable. At the very least, the minimum number of TOF digis for triggering should always be >2, to avoid "T0 only" triggers.

Dear @d.smith,

you have been identified as code owner of at least one file which was changed with this merge request.

Please check the changes and approve them or request changes.

added CodeOwners label

mentioned in merge request !694 (closed)

added 24 commits

• b7326f60...01e5e5ae - 23 commits from branch computing:master
• 0d955f09 - Modified CbmBuildEventsQa and QA function in CbmSeedFinderSlidingWindow to...

Compare with previous version

merged