Fix Performance issue in GroupDetectors (#39246)

* reserve space for event list

* fix variable

* add release note

* add timing and instrument

Author: searscr

Framework/DataHandling/src/GroupDetectors2.cpp

@@ -1010,6 +1010,13 @@ size_t GroupDetectors2::formGroupsEvent(const DataObjects::EventWorkspace_const_
     // Start fresh with no detector IDs
     outEL.clearDetectorIDs();
 
+    // Reserve space for the event list
+    std::size_t totalEvents = 0;
+    for (auto i : it->second) {
+      totalEvents += inputWS->getSpectrum(i).getNumberEvents();
+    }
+    outEL.reserve(totalEvents);
+
     // the Y values and errors from spectra being grouped are combined in the
     // output spectrum
     // Keep track of number of detectors required for masking

docs/source/release/v6.13.0/Framework/Algorithms/Bugfixes/39246.rst

@@ -0,0 +1 @@
+- Fix issue with :ref:`GroupDetectors <algm-GroupDetectors-v2>` where the algorithm would freeze on large datasets while adding EventList to the Output Workspace. Testing with TOPAZ-50006 execution time was reduced from 2+ hours to ~30 seconds.