Skip to content
Snippets Groups Projects

Add test to see if any events pass the pipeline

Merged Add test to see if any events pass the pipeline
dguest/hh4b-analysis:add-pass-check into master
All threads resolved!
Merged Dan Guest requested to merge dguest/hh4b-analysis:add-pass-check into master
All threads resolved!

This adds a test to see if any events pass a CI pipeline.

I had to do some less pretty things:

  • Most files were failing because of the truth selection, so I turned the filter there off. Apparently @viruelas is working on fixing that.
    • I also turned off the ntuple outputs, since the truth particles were being added conditional on the filter passing. In events where nothing was added ntuple writing would fail.
  • I remove the trigger requirement for data. It's sort of a hack until we have a data sample where more triggers pass (see #32 (closed)).

While getting the data tests running I noticed that the nutple writer was asking for truth info while looking at data, so I fixed that too. Data is also missing Tau4_wta I'm guessing it was just not saved in those ntuples.

Edited by Dan Guest

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
  • Dan Guest resolved all threads

    resolved all threads

  • Dan Guest added 1 commit

    added 1 commit

    • 1d528fe8 - remove thing that did nothing

    Compare with previous version

    • Teng Jian Khoo
      Teng Jian Khoo @khoo
      Owner
      Resolved by Teng Jian Khoo

      Awesome. I think there is probably some way we could get the cutflow from book keepers or CutFlowSvc rather than needing a separate alg, but since this works and is super useful, let's go with it for now and maybe check if we already have that info floating around or need to actually hook up the filter algs to something.

  • Teng Jian Khoo enabled an automatic merge when the pipeline for 1d528fe8 succeeds

    enabled an automatic merge when the pipeline for 1d528fe8 succeeds

  • Dan Guest added 1 commit

    added 1 commit

    Compare with previous version

  • Teng Jian Khoo resolved all threads

    resolved all threads

  • Teng Jian Khoo merged

    merged

  • Teng Jian Khoo mentioned in commit b9073144

    mentioned in commit b9073144

  • Louis D'Eramo added Code:Pipeline label

    added Code:Pipeline label

    • Please register or sign in to reply