Skip to content
Snippets Groups Projects

Component accumulator in DAODs - part 3a: common physics augmentations

Closed James Catmore requested to merge jcatmore/athena:master-DAOD_CA_part3a into master

This large merge request is a migration of the tracking, muon and e-gamma common augmentations applied and saved in DAODs, including PHYS and PHYSLITE, to the component accumulator. It also provides a migrated version of PHYS. Since every tool used needs a new configuration fragment it is necessary to create a number of new files containing these config methods. A few packages that didn't previously have python directories now have python configs so this has required adding an extra line to the CMakeFiles so that the modules can be found. However, since this is completely new python and only runs with the new Derivation_tf transform, it does not impact any existing DAODs built using Reco_tf.

Concretely the following has been added:

  • new configs for all of the inner detector track augmentation tools
  • new configs for all of the muon augmentation tools
  • new configs for all of the e-gamma augmentation tools

These are all summoned from the PhysCommonConfig fragment, which itself is called from PHYS/PHYSLITE and any other formats that need to use it. The truth setup is directly in PhysCommonConfig.

As things stand the following are missing:

  • migration of jets
  • taus/flavour tagging
  • trigger matching information

Without these last three items the version of PHYS produced by this MR clearly isn't yet usable for physics, but can be validated for the tracking, muon and e-gamma domains. For this reason the MR is being put in for merging now, to avoid it becoming too large, since the remaining work will probably involve touching at least as many files again.

The following items are marked "TODO" because it isn't clear yet how to deal with them in the component accumulator and/or ConfigFlags:

  • InDetFlags are being used in the inner detector configuration. With what should these be replaced? Specifically AddPseudoTracks, DecoLRTTTVA and MergeLRT
  • How should we handle the IsoTrackCones in the muon configuration?
  • How should we correctly determine and then flag the simulation flavour?
  • TruthMetaDataWriter fails due to missing /TagInfo

Consequently there are blocks of commented python corresponding to these items, which need to remain until we have solution (at which point they can be uncommented and migrated). I'll create JIRAs for these over the coming days.

Finally, the thinning machinery appears to be broken in CA (including in the example TEST3). I'll follow this up separately.

Edited by James Catmore

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
  • James Catmore added 2 commits

    added 2 commits

    • f2bbe60e - Adding muon config; making tools that are configured outside of the derivation framework private
    • 7b99675e - Merge branch 'master-DAOD_CA_part3a' of...

    Compare with previous version

  • James Catmore resolved all threads

    resolved all threads

  • James Catmore changed the description

    changed the description

  • James Catmore added 1 commit

    added 1 commit

    Compare with previous version

  • James Catmore changed title from Draft: Component accumulator in derivation framework - part 3a: common physics... to Draft: Component accumulator in DAODs - part 3a: common physics augmentations

    changed title from Draft: Component accumulator in derivation framework - part 3a: common physics... to Draft: Component accumulator in DAODs - part 3a: common physics augmentations

  • James Catmore changed the description

    changed the description

  • James Catmore changed the description

    changed the description

  • Christos Anastopoulos
  • James Catmore added 1 commit

    added 1 commit

    • a21b176c - Passing all config fragments the ConfigFlags as per guidelines. Renaming inconsistently named file.

    Compare with previous version

  • James Catmore added 1 commit

    added 1 commit

    • 9e1d2d2c - Using ConfigFlags.Sim.SimBarcodeOffset

    Compare with previous version

  • James Catmore changed the description

    changed the description

  • James Catmore changed the description

    changed the description

  • James Catmore added 1 commit

    added 1 commit

    • 7a927c25 - Switching some of the ID flags to use ConfigFlags

    Compare with previous version

  • James Catmore added 1 commit

    added 1 commit

    • 4e076c73 - Adding common truth contents

    Compare with previous version

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