Removed functions findByName and findByID from the MoveInstrumentComponent and RotateInstrumentComponent and replace calls of them by appropriate member functions of instrument. re #3709

Refs #3859: SliceMD algorithm. Lots of tests in the SlicingAlgorithmTest class to try every possibility. Better implicit function for non-axis-aligned cuts. Still not fully general, need to add non-axis-aligned 4D cuts and some reduced-dimension cuts.

Allow for 'neutronic' detectors to have a different shape to their 'physical' counterpart. Re #1944.

Although not related to this ticket here just a minor change (added example bounding box) to one of the IDF unit testing files. re #3856

Fixes #3861: SlicingAlgorithm class will be shared by 2 algos. Refs #3859: Start of SliceMD algorithm.

Fixes #3773: event -> HKL conversion now works well. In TOPAZ_3131 (see sample diffraction workflow script) the peaks look perfectly aligned with the axes.

Fixes #3772: QSample of predicted peaks is corrected. Also tweaked prediction and allowed using any workspace type as input.

Fix error Refs #3858. Error is that ADSWorspaceProvider is templated by the type of workspace that it's meant to check for/against. Previous commit had made this object commong to both Presenter types.

Refs #3710. B-matrix is now extracted, but there is not currently a placeholder on the IMDEventWorkspace for this. We will likely need knowledge of the oriented lattice on IMDEventWorkspaces in the future.