Merge pull request #17387 from mantidproject/17082_compactMD_out_of_bounds

Fix CompactMD out of bounds bug

Framework/MDAlgorithms/src/IntegrateMDHistoWorkspace.cpp

@@ -147,13 +147,18 @@ void setMinMaxBins(Mantid::coord_t &pMin, Mantid::coord_t &pMax,
   snappedPMax += offset;
   snappedPMin += offset;
 
-  if (pMin != snappedPMin) {
+  if (snappedPMin < dimension->getMinimum()) {
+    snappedPMin = dimension->getMinimum();
+  } else if (pMin != snappedPMin) {
     std::stringstream buffer;
     buffer << "Rounding min from: " << pMin
            << " to the nearest whole width at: " << snappedPMin;
     logger.warning(buffer.str());
   }
-  if (pMax != snappedPMax) {
+
+  if (snappedPMax > dimension->getMaximum()) {
+    snappedPMax = dimension->getMaximum();
+  } else if (pMax != snappedPMax) {
     std::stringstream buffer;
     buffer << "Rounding max from: " << pMax
            << " to the nearest whole width at: " << snappedPMax;

Framework/MDAlgorithms/test/CompactMDTest.h

@@ -216,7 +216,8 @@ public:
                       outputWorkspace->getDimension(0)->getBinWidth(),
                       inWS->getDimension(0)->getBinWidth());
   }
-  void test_compact_md_throws_when_loading_empty_workspace() {
+
+  void test_compact_md_does_not_throw_when_loading_empty_workspace() {
     using namespace Mantid::DataObjects;
     const size_t numDims = 1;
     const double signal = 0.0;