Skip to content
Snippets Groups Projects

Enable scheduling of reco jets in IDPVM standalone

This MR enables the scheduling of reco small-R jet finding in standalone IDPVM jobs if these jets are required by the IDPVM run (e.g. sum(pt,w) ) and not present. Based on work by @tstreble.

May need some cleaning up / tuning. CC @gfrattar @pbhattar @kostyuk @keli @tstreble @vcairo @gfacini @sroe @goetz in case of comments.

Edited by Maximilian Emanuel Goblirsch-Kolb

Merge request reports

Pipeline #3744199 passed

Pipeline passed for 9798c190 on goblirsc:AddRecoJetsIDPVM

Approval is optional

Merged by Frank WinklmeierFrank Winklmeier 3 years ago (Mar 22, 2022 6:16pm UTC)

Merge details

  • Changes merged into master with c0de75f0 (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
  • added 1 commit

    • 25ddf1f9 - update The Copyright Message to avoid L1 trouble

    Compare with previous version

  • added 165 commits

    • 25ddf1f9...0e21d686 - 164 commits from branch atlas:master
    • 4011fc7c - Merge remote-tracking branch 'upstream/master' into AddRecoJetsIDPVM

    Compare with previous version

  • added 3 commits

    Compare with previous version

  • resolved all threads

  • Maximilian Emanuel Goblirsch-Kolb marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 2 packages:

    • InnerDetector/InDetRecTools/InDetHardScatterSelectionTool
    • InnerDetector/InDetValidation/InDetPhysValMonitoring

    Affected files list will not be printed in this case

    Adding @goetz ,@amorley ,@sroe as watchers

  • Reverted to AntiKt4EMTopoJets after consultation with experts (thanks @mswiatlo for the useful feedback!), solved the philosopher, this MR is ready to go

  • Amazing stuff. Thank you @goblirsc

  • the code looks fine from L1 perspective. Once the above threads are resolved, it should be approved.

    Roman(L1)

  • added 1 commit

    • 61c790e9 - trying to satisfy jenkins. Delaying a MR 24h on a copyright message seems...

    Compare with previous version

  • This merge request affects 2 packages:

    • InnerDetector/InDetRecTools/InDetHardScatterSelectionTool
    • InnerDetector/InDetValidation/InDetPhysValMonitoring

    Affected files list will not be printed in this case

    Adding @goetz ,@amorley ,@sroe as watchers

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