Merge pull request #23705 from mantidproject/23697_peak_numbers_combined

Peak numbers unique when peak workspaces combined

Framework/Crystal/src/SaveIsawPeaks.cpp

@@ -267,7 +267,10 @@ void SaveIsawPeaks::exec() {
   runMap_t::iterator runMap_it;
   for (runMap_it = runMap.begin(); runMap_it != runMap.end(); ++runMap_it) {
     // Start of a new run
-    appendPeakNumb += maxPeakNumb;
+    if (maxPeakNumb > 0) {
+      appendPeakNumb += maxPeakNumb + 1;
+      maxPeakNumb = 0;
+    }
     int run = runMap_it->first;
     bankMap_t &bankMap = runMap_it->second;
 

docs/source/release/v3.14.0/diffraction.rst

@@ -41,6 +41,7 @@ Bugfixes
 - :ref:`CentroidPeaksMD <algm-CentroidPeaksMD>` now updates peak bin counts.
 
 - :ref:`FindPeaksMD <algm-FindPeaksMD>` now finds peaks correctly with the crystallography convention setting and reduction with crystallography convention is tested with a system test.
+- :ref:`SaveIsawPeaks <algm-SaveIsawPeaks>` does not have duplicate peak numbers when saving PeaksWorkspaces with more than one RunNumber.
 
 Powder Diffraction
 ------------------