Re #24314 fix unit test, add release note

Framework/DataObjects/test/PeakTest.h

@@ -239,9 +239,9 @@ public:
   void test_samplePos() {
     Peak p(inst, 10000, 2.0);
     p.setSamplePos(1.0,1.0,1.0);
-    TS_ASSERT_EQUALS(p.getHKL(), V3D(1.0, 1.0, 1.0));
+    TS_ASSERT_EQUALS(p.getSamplePos(), V3D(1.0, 1.0, 1.0));
     p.setSamplePos(V3D(2.0,2.0,2.0));
-    TS_ASSERT_EQUALS(p.getHKL(), V3D(2.0, 2.0, 2.0));
+    TS_ASSERT_EQUALS(p.getSamplePos(), V3D(2.0, 2.0, 2.0));
   }
 
   void test_getBank_and_row() {

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

@@ -49,6 +49,8 @@ Bugfixes
 - :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.
 
+- :ref:`OptimizeCrystalPlacement <algm-OptimizeCrystalPlacement>` now updates the sample location used by peaks.  Previously, the sample was effectively left unmoved.
+
 Powder Diffraction
 ------------------