Skip to content

Add HLT2 test for all lines and particle makers, tidy up D02HH line module

Alex Pearce requested to merge apearce-d02hh-cleanup into master

Seeing as the D02HH module has been used as a 'guide' for line authors, we should make sure it represents current best practices. I've added a bit more documentation (using the numpydoc convention in functions) and removed the default None arguments, which !185 (merged) negated the need for.

Also added two tests:

  1. Runs all HLT2 lines on reconstructed MC, as suggested by @rmatev. This run over 100 events from a minimum bias MC sample, and checks the logfile to see if any lines fire. The B_{s} \to J/\psi\phi line, introduced in !177 (merged), fires on two events, so the test currently fails. We should come up with a more robust check.
  2. Runs all particle makers as individual HLT2 lines on reconstructed MC. This highlighted some missing makers, so I've added those, and I removed the T-track particle makers because they produce no output (#75 (closed)).
Edited by Alex Pearce

Merge request reports