Skip to content

MT-safe migration for FastHitConvertTool

Benjamin Michael Wynne requested to merge bwynne/athena:MTsafeFastHitTool into master

Currently FastHitConvertTool gives an error if run in AthenaMT, because its public method (process) casts off the constness of the tool itself in order to run. However, this is all to support historical structure of the tool, and there was nothing fundamentally blocking a migration to AthenaMT. Used DataHandleKeys to allow fully const execution of the tool, except in one instance (PileupEventInfo is retrieved and modified by non-const eventStore()->retrieve), and the error is moved there.

Checked that this does not change results in ATLFASTII or FastCaloSimV2 (single threaded). Allows FastCaloSim to run multithreaded without crashing.

Needed for https://its.cern.ch/jira/browse/ATLASSIM-4764

Tagging @jchapman

Merge request reports