Use all runs in FindPeaksMD

[Vickie Lynch]

From SNS user: After I merge md workspace using the following command, I obtained the mdqsamp.

mdqsamp=MergeMD(mdqsampparts)

If I use FindpeaksMD to get peaks, only the first run of all merged runs is used to find peak. Is it a bug in the algorithm? Ideally we should get peaks from all runs.

Line from FindPeaksMD that should loop over runs:

  ExperimentInfo_sptr ei = ws->getExperimentInfo(0);

[Vickie Lynch]

From Andrei: I am just wondering if there are contributions from two workspaces to the same peak, what goniometer setting should we use. I think the algorithm uses the peaks from different workspaces, just converts Q sample to Q lab using the first goniometer.

[Andrei Savici]

This approach will fail to record peaks for which it cannot find a detector with the first goniometer settings.

[Vickie Lynch]

Refs #11004 find peaks from all runs that can be mapped to a detector

Changeset: 150a8635b8f624951cca6295c719004a28aa0672

[anonymous]

This is being verified as ​pull request #176.

[Vickie Lynch]

Refs #11004 fix unit test

Changeset: 7846a35649622a242f2f3850b62d3957260812a9

[Vickie Lynch]

Refs #11004 fix warning in test

Changeset: 49d8f45c3f802625cb3b1f7cffec701dab674f15

[anonymous]

This is being verified as ​pull request #183.

[anonymous]

This is being verified as ​pull request #192.

[Vickie Lynch]

Refs #11004 fix windows warning in test

Changeset: 75a35361a4d94971ca52a3bed41ef2ff6424a32f

[anonymous]

retest this please

[Ross Whitfield]

Merge pull request #192 from mantidproject/11004_FindPeaks_all_runs

11004 find peaks all runs

Full changeset: ​cd1f59324251bc4817c9e979ec63ff8ccd738602

[Nick Draper]

Somehow these slipped through without a resolution. Set to Fixed.

[Stuart Campbell]

This ticket has been transferred to github issue ​11843