Skip to content

Optimize TriggerProducer

Kevin Pedro requested to merge github/fork/kpedro88/Run2_2017 into Run2_2017

TriggerProducer was in the top 5 in a CPU test with the 2017 scenario. Hash tables make O(N^2) into O(N)! For the current list of triggers (containing both 2016 and 2017 lines), this is a ~10x speedup.

Also, we had been storing the version number in the TriggerName string if the trigger was found in the event. Instead, we can store the version number in a separate branch, so the TriggerName branch is always the same (in case of future optimizations for storing metadata).

Fixed some bugs in the trigger name list revealed by these changes and added a comment to clarify the details.

Merge request reports