Skip to content
Snippets Groups Projects

Updating PLIT input variables

Merged Simone Francescato requested to merge sfrances/athena:main into main
All threads resolved!

This MR updates the list of variables used by PLIT, after the upload in cvmfs of the new tagger model (ATLASG-2865).

The models are available only for Run2, but I included dummy Run2 configs.

I ran some test MUON5 derivation to see if all is ok

# data18
Derivation_tf.py --inputAODFile /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/CampaignInputs/data18/AOD/data18_13TeV.00357772.physics_Main.merge.AOD.r13286_p4910/1000events.AOD.27655096._000455.pool.root.1 --formats MUON5 --outputDAODFile test --maxEvents 100
# mc20
Derivation_tf.py --inputAODFile /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/CampaignInputs/mc20/AOD/mc20_13TeV.410470.PhPy8EG_A14_ttbar_hdamp258p75_nonallhad.merge.AOD.e6337_e5984_a907_r14860_r14919/1000events.AOD.34805450._000733.pool.root.1 --formats MUON5 --outputDAODFile test --maxEvents 100

For the sake of testing, I also tried to force Run3 MC to run with Run2 configs and it's anyway ok

Derivation_tf.py --inputAODFile /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/CampaignInputs/mc23/AOD/mc23_13p6TeV.601229.PhPy8EG_A14_ttbar_hdamp258p75_SingleLep.recon.AOD.e8514_s4159_r14799/1000events.AOD.34124794._001345.pool.root.1 --formats MUON5 --outputDAODFile test --maxEvents 100 --preExec "from AthenaConfiguration.Enums import LHCPeriod; flags.GeoModel.Run = LHCPeriod.Run2”

I haven't found any critical issues at surface level. Larger scale validation of the tool is on-going in the IFF group.

@kristin @ahabouel @lucam @sangelid @mvanadia @falonso @emaricic

Edited by Simone Francescato

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
  • Simone Francescato mentioned in merge request !78285 (merged)

    mentioned in merge request !78285 (merged)

  • Simone Francescato resolved all threads

    resolved all threads

  • Please register or sign in to reply
    Loading