Skip to content
Snippets Groups Projects

DAOD_TRIG prototype for Run 3

Merged Oleg Kuprash requested to merge okuprash/athena:DAOD_TRIG_prototype into master
All threads resolved!

Starting with copying PHYS.py to TRIG.py and enabling storing trigger containers. Adding some first trigger collections from Run 3 EDM: new navigation, b-jet, and B-tagging containers (via smart slimming list).

Checked using an AOD file produced in master with Run 3 trigger from /eos/atlas/atlascerngroupdisk/data-art/build-output/master/Athena/x86_64-centos7-gcc8-opt/2020-09-21T2101/TrigAnalysisTest/test_trigAna_RDOtoAOD_v1Dev_build/AOD.pool.root

and running the transform

Reco_tf.py --inputAODFile AOD.pool.root --outputDAODFile art.pool.root --reductionConf TRIG  --preExec "from AthenaCommon.DetFlags import DetFlags;DetFlags.detdescr.all_setOff(); DetFlags.BField_setOn(); from AthenaCommon.AlgSequence import AlgSequence;topSequence = AlgSequence ();topSequence += CfgMgr.xAODMaker__DynVarFixerAlg(\"BTaggingELFixer\", Containers = [\"BTagging_AntiKt4EMTopoAux.\" ] )" --postExec "from AthenaCommon.AppMgr import ServiceMgr;ServiceMgr.MessageSvc.defaultLimit=0;ServiceMgr.MessageSvc.debugLimit=0" --debug

In this test, the trigger navigation, b-jets and B-tagging containers are stored in the output DAOD.
Tagging @dguest for planning the DAOD production for b-jet trigger optimization.
Tagging @jburr as this is potentially related to trigger for analyses (including trigger containers for matching to offline).

Merge request reports

Pipeline #1972355 passed

Pipeline passed for a3bfedd0 on okuprash:DAOD_TRIG_prototype

Approval is optional

Merged by Adam Edward BartonAdam Edward Barton 4 years ago (Sep 29, 2020 2:39pm UTC)

Merge details

  • Changes merged into master with 405a44a8 (commits were squashed).
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • :white_check_mark: CI Result SUCCESS (hash 421cfe2a)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21108]

  • :negative_squared_cross_mark: CI Result FAILURE (hash 9674803f)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :o: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :o: Athena: number of compilation errors 1, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21105]

    • Contributor
      Resolved by Jon Burr

      Hi @okuprash,

      Sorry for being late in getting to look at this.
      This is only expected to have the online b-jets in it right?
      We would not be able to do online-offline matching with this (except for b-jets) but it's not clear to me if it's intended that we could?

      Cheers,
      Jon

  • Oleg Kuprash added 1 commit

    added 1 commit

    Compare with previous version

  • This merge request affects 2 packages:

    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkCore
    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkTrigger
  • Oleg Kuprash resolved all threads

    resolved all threads

  • Dan Guest mentioned in merge request !36811 (merged)

    mentioned in merge request !36811 (merged)

  • :white_check_mark: CI Result SUCCESS (hash a3bfedd0)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 21153]

  • Santiago Noacco Rosende resolved all threads

    resolved all threads

  • mentioned in commit 405a44a8

  • Please register or sign in to reply
    Loading