Skip to content
Snippets Groups Projects
Commit d293e14b authored by Tim Martin's avatar Tim Martin
Browse files

use isMasterSlot call

parent 394370e8
6 merge requests!58791DataQualityConfigurations: Modify L1Calo config for web display,!46784MuonCondInterface: Enable thread-safety checking.,!46776Updated LArMonitoring config file for WD to match new files produced using MT,!45405updated ART test cron job,!42417Draft: DIRE and VINCIA Base Fragments for Pythia 8.3,!34568TrigCost updates for T11
...@@ -73,7 +73,7 @@ uint32_t CostData::onlineSlot() const { ...@@ -73,7 +73,7 @@ uint32_t CostData::onlineSlot() const {
return m_slot; return m_slot;
} }
bool CostData::isMaterSlot() const { bool CostData::isMasterSlot() const {
return (onlineSlot() == 0); return (onlineSlot() == 0);
} }
......
...@@ -89,7 +89,7 @@ class CostData { ...@@ -89,7 +89,7 @@ class CostData {
/** /**
* @return True if event was processed in the master slot (0), and hence contains cost data spanning all concurrent slots * @return True if event was processed in the master slot (0), and hence contains cost data spanning all concurrent slots
*/ */
bool isMaterSlot() const; bool isMasterSlot() const;
/** /**
* @brief Getter of the total algorithm CPU time in the event. * @brief Getter of the total algorithm CPU time in the event.
......
...@@ -155,7 +155,7 @@ StatusCode TrigCostAnalysis::execute() { ...@@ -155,7 +155,7 @@ StatusCode TrigCostAnalysis::execute() {
ATH_CHECK( range->newEvent( costData, getWeight(context) ) ); ATH_CHECK( range->newEvent( costData, getWeight(context) ) );
if (checkDoFullEventDump(context, onlineSlot)) { if (checkDoFullEventDump(context, costData)) {
ATH_CHECK( dumpEvent(context) ); ATH_CHECK( dumpEvent(context) );
} }
...@@ -163,8 +163,8 @@ StatusCode TrigCostAnalysis::execute() { ...@@ -163,8 +163,8 @@ StatusCode TrigCostAnalysis::execute() {
} }
bool TrigCostAnalysis::checkDoFullEventDump(const EventContext& context, const uint32_t onlineSlot) { bool TrigCostAnalysis::checkDoFullEventDump(const EventContext& context, const CostData& costData) {
if (onlineSlot == 0 if (costData.isMasterSlot()
and m_fullEventDumps < m_maxFullEventDumps and m_fullEventDumps < m_maxFullEventDumps
and context.eventID().event_number() % m_fullEventDumpProbability == 0) and context.eventID().event_number() % m_fullEventDumpProbability == 0)
{ {
......
...@@ -116,11 +116,10 @@ class TrigCostAnalysis: public ::AthHistogramAlgorithm { ...@@ -116,11 +116,10 @@ class TrigCostAnalysis: public ::AthHistogramAlgorithm {
/** /**
* @brief Check if event dumping should be performed for the current event. * @brief Check if event dumping should be performed for the current event.
* @param[in] context Event context. * @param[in] context Event context.
* @param[in] onelineSlot The slot used to process the event in the trigger. * @param[in] costData We only dump the master-slot (slot:0) as these events cary payload data for all slots / all threads.
* We only dump slot 0 as these events cary payload data for all events.
* @return True, if the event should be dumped. * @return True, if the event should be dumped.
*/ */
bool checkDoFullEventDump(const EventContext& context, const uint32_t onlineSlot); bool checkDoFullEventDump(const EventContext& context, const CostData& costData);
/** /**
* @brief Return or construct and return a Range for the Context. Might return nullptr range. * @brief Return or construct and return a Range for the Context. Might return nullptr range.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment