Skip to content
Snippets Groups Projects

Add a rerun HLT2 option in Moore

Merged Kara Mattioli requested to merge add_HLT2_rerun_option_in_Moore into master

Update: After discussions on the best way to configure the bankKillers (see discussions below), it was decided to remove the rerun_hlt2 flag and instead create an input_process option for Moore. The default setting is input_process = 'Hlt1'. When input_process = 'Hlt2' is specified in the options file and HLT2 is being run, the bankKillers are configured to remove the DstData and Hlt2DecReport banks from the previous HLT2 processing.

The Options file has been updated to provide an example of setting the input_process option.

This updated MR now depends on LHCb!4017 (merged)

Original MR (see update above for details of the changes) This MR adds a rerun_hlt2 flag in the Moore config file to be set to True when running over HLT2 passthrough data. When this flag is set to True, the DstData and Hlt2DecReport banks from the passthrough run are deleted and replaced with the banks produced in the HLT2 re-run. This is needed in order for DaVinci to be able to process the DST produced from re-running HLT2.

Files added:

  • Options file to re-run HLT2 with the flag rerun_hlt2 = True
  • qmtest to test the HLT2 re-run

In other words, this MR applies the fix discussed in #524

Edited by Kara Mattioli

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
  • Sascha Stahl
  • Thanks a lot, @kmattiol .

  • Kara Mattioli added 1 commit

    added 1 commit

    • a1f8d1d0 - Remove output level=2 in Moore config

    Compare with previous version

  • Kara Mattioli added 1 commit

    added 1 commit

    • d36135b0 - Remove commented RawBanksToIgnore

    Compare with previous version

  • mentioned in issue #540 (closed)

  • Sascha Stahl mentioned in merge request LHCb!4011 (closed)

    mentioned in merge request LHCb!4011 (closed)

  • Kara Mattioli added 19 commits

    added 19 commits

    Compare with previous version

  • Kara Mattioli added 1 commit

    added 1 commit

    • 2978f5bd - Configure bankKiller with input_process flag instead of rerun_hlt2 flag

    Compare with previous version

  • Kara Mattioli added 1 commit

    added 1 commit

    Compare with previous version

  • Kara Mattioli mentioned in merge request LHCb!4017 (merged)

    mentioned in merge request LHCb!4017 (merged)

  • Sebastien Ponce resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading