Skip to content
Snippets Groups Projects

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

Merged 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading