Skip to content
Snippets Groups Projects

Adjust ICaloFutureElectron to make CaloFutureElectron tool thread safe

Merged Adam Szabelski requested to merge MakeCaloFutureElectronThreadSafe into master
All threads resolved!

This MR is connected to Rec!1806 (merged) and refers to issue Rec#106
All callers need to be separate from set and accept protoParticle as argument.
set is left as public to keep the logic from the calling algorithms, especially when a counter depends on successful set

Edited by Rosen Matev

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
  • Adam Szabelski mentioned in merge request Rec!1806 (merged)

    mentioned in merge request Rec!1806 (merged)

  • Adam Szabelski changed the description

    changed the description

  • Adam Szabelski added 1 commit

    added 1 commit

    • e8f6acbb - Apply suggestion to CaloFuture/CaloFutureInterfaces/CaloFutureInterfaces/ICaloFutureElectron.h

    Compare with previous version

  • Adam Szabelski added 2 commits

    added 2 commits

    • bf781ba5 - Change naming: 'set'-> 'getElectronBrem'
    • f0ddb00b - Implement structure hypoPairStruct

    Compare with previous version

  • Adam Szabelski resolved all threads

    resolved all threads

  • added Calo label

  • Sascha Stahl unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Adam Szabelski resolved all threads

    resolved all threads

  • Edited by Software for LHCb
  • Marian Stahl changed the description

    changed the description

  • Marian Stahl assigned to @mstahl and unassigned @aszabels

    assigned to @mstahl and unassigned @aszabels

  • Marian Stahl added 1 deleted label

    added 1 deleted label

  • Throughputs look good to me:
    HLT2 baseline (including Moore!349 (merged))
    HLT2 brunelesque is stable: today yesterday

  • Marian Stahl mentioned in merge request Rec!1833 (merged)

    mentioned in merge request Rec!1833 (merged)

  • Marian Stahl approved this merge request

    approved this merge request

  • Marian Stahl assigned to @rmatev and unassigned @mstahl

    assigned to @rmatev and unassigned @mstahl

  • Rosen Matev changed title from make CaloFutureElectron tool thread safe. Adjust ICaloFutureElectron to Adjust ICaloFutureElectron to make CaloFutureElectron tool thread safe

    changed title from make CaloFutureElectron tool thread safe. Adjust ICaloFutureElectron to Adjust ICaloFutureElectron to make CaloFutureElectron tool thread safe

  • Rosen Matev removed 1 deleted label

    removed 1 deleted label

  • unassigned @rmatev

  • merged

  • Rosen Matev mentioned in commit 236b1e28

    mentioned in commit 236b1e28

  • mentioned in issue Moore#128 (closed)

  • Please register or sign in to reply
    Loading