This reverts commit 21f52c67 and fixes it.

refs #6331

The test is currently commented out, since the test data file
is too large to add to github (28 MB).  Also removed a few
debug prints.

refs #6331

Basically complete and hopefully correct.  Still need to do
some more testing and make a test.
refs #6331

A new BASIS file showed up a thread-safety issue with each bank
loading task trying to modify the same eventVector. It will
require monitoring the performance though as the repeated mutex
lock/unlock may cause performance issues.

This reverts commit ed408067 and fixes
the system tests until further investigation of the failure (and also
why the unittests passed!).

refs #6331

refs #6331

For this to work from a PeaksWorkspace loaded from a file
we need to calculate a UB matrix.  We assume that the
file contains INDEXED peaks, and calculate the UB from
the indexes and Lab-Q vectors.  Un-indexed peaks still
need to be skipped.
refs #6331

This reverts commit 1b515d41.

It was calling cut() recursively.

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

Replaces copy-and-pasted code.

This is not yet complete and correct.
refs #6331

Added check in toInstrumentAndNumber() method.