Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 122
    • Merge requests 122
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !54413

Support large-R UFO jets in PHYS derivation

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Pierre Antoine Delsart requested to merge delsart/athena:ufoDeriv22 into master Jun 20, 2022
  • Overview 16
  • Commits 4
  • Pipelines 2
  • Changes 11

This MR brings large-R UFO jets building to PHYS derivations.

This has been tested with :

Derivation_tf.py --CA --inputAODFile /eos/atlas/atlascerngroupdisk/perf-jets/mc20_13TeV.410470.PhPy8EG_A14_ttbar_hdamp258p75_nonallhad.recon.AOD.e6337_s3681_r13144/AOD.27120891._005893.pool.root.1 --outputDAODFile pool.root --formats PHYS

and

 Reco_tf.py --inputAODFile /eos/atlas/atlascerngroupdisk/perf-jets/mc20_13TeV.410470.PhPy8EG_A14_ttbar_hdamp258p75_nonallhad.recon.AOD.e6337_s3681_r13144/AOD.27120891._005893.pool.root.1 --outputDAODFile pool.root --reductionConf PHYS --maxEvents 100 --postExec='from DerivationFrameworkJetEtMiss.JetCommon import swapAlgsInSequence;swapAlgsInSequence(topSequence,"jetalg_ConstitModCorrectPFOCSSKCHS_GPFlowCSSK", "UFOInfoAlgCSSK" )'

Note the Reco_tf.py only works with a postExec statement because the new style config used in jet is not working smoothly with the old-style config used by this Reco_tf.py command. I'm not sure there's a simpler way than using a postExec. Probably @jcatmore can tell if this is acceptable ?

Also tagging @cdelitzs @mswiatlo @camacho @mspalla @haweber

Edited Jun 20, 2022 by Pierre Antoine Delsart
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: ufoDeriv22