Avoid std::map in EventSelectionModule#16396
Conversation
|
@ddobrigk can you have a look here? Looking up in a map is very expensive due to the pointer traversal and constructing it requires one allocation per node which is also expensive. A binary search in a vector of pairs should be much faster. Also, @aalkin was suggesting to use a index table for this? |
|
@ekryshen can you have a look at this? |
|
@ktf Thanks, looks reasonable. Actually I suspect that we don't need this map or vector of pairs any longer. It was introduced as a workaround for early passes of pp2022 where trigger info wasn't aligned with actual bcs. As far as I understand, the trigger functionality is used by EMCAL analysers only. I will check with them if they still need support of old passes. If not, we can simply remove this workaround. |
No description provided.