Refs #22197

- put each declaration of a variable in a single line
- using const auto  & to get x, y, e, dx
- directly set the size and xerrors in setPointStandardDeviations

Refs #22218

Refs #22197

- shorter tests by using std::vector equality check capacity
- add test for !hasDx capability

Refs #22218

(cherry picked from commit 37621330)

(cherry picked from commit da3f0d8f)

(cherry picked from commit a2fca669)

(cherry picked from commit 08b308ba)

Refs #22218

Refs #22218

Refs #22218

Refs #22218

- in WorkspaceCreationHelper: do not overload create2DWorkspaceBinned but use new method create2DWorkspaceNonUniformlyBinned

Refs #22218

- documentation modified (description, usage exmple)
- clang
- performance test ConjoinXRunsTest (<6 sec with dx, without dx it would be 0.1 sec faster)
- fix ConjoinXRuns (bracket did not close correcly)
- add unit test without dx values in ConjoinXRuns (all other tests run with dx)

Refs #22218

(cherry picked from commit f5128b18)

(cherry picked from commit f9c7f9e6)

I could have added something like

    Counts counts{{4, 9, 16}};
    Points points{{0.4, 0.9, 1.1}};
    ws6->setHistogram(3, points, counts);
    for (auto i = 0; i < 5;
         i++) { // modify all 5 spectra in terms of Dx
      ws6->setPointStandardDeviations(i, 3);
      auto dx = Mantid::Kernel::make_cow<HistogramDx>(3, 8.8);
      ws6->setSharedDx(i, dx);
    }

But the current solution seems to be easier since I do not want to check Dx again here; if algorithm executed and did not throw is enough to test (->RunCombinationHelper).

Refs #22218

- now 90 percent of the ConjoinXRunsTest pass

Refs #22218

- I thought .rawData() is only for histogram internal use
- ConjoinXRunsTest passes now, but dx is switched off

Refs #22218

- Two unit tests are failing for me: ConjoinXRunsTest and MergeRunsTest (different values event workspace??)

Refs #22218

Refs #22218

Refs #22218

Refs #22218

Refs #22218

- unit test passes since use of dx uncommented

Refs #22218

- CopyInstrumentParametersTest.h included WorkspaceCreationHelperTest.h -> resolved this situation and could delete unused includes

Refs #22218

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

re #22143

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>