The workspace spectra are now modified directly.
Refs #9435

...with setting of detector ID directly in ISpectrum.

The Algorithms package is now free of any mention of the SpectraDetectorMap.

Thisgs to note:
 * C2450 warnings have been suppressed when including Poco headers
   that inherit from std::stream
 * Boost date_time no longer accepts just a date for an ISO date/time
   string

by using the ADS::retrieveWS<Type>() method.

Python script changes to follow.

Add pointer to physical instrument as a member (not used in vast majority of cases) of Instrument. Show this 'physical instrument' in the instrument view, if it exists. Re #1944.
P.S. This again lead to a const change which accounts for most of the changes.

Add a copy constructor to Instrument. Oh, and go on a const-correctness hunt that led this commit to touch slightly more than just the one class. Re #1944.

Eradicate IInstrument. Instrument's the one to talk to now (it was the only descendent class anyway). Re #2041.

- Refs #3203: Implemented ISpectrum. It is the base class of EventList and Histogram and contains the spectrum number and list of detector IDs for a given spectrum.
- ISpectrum return X/Y/E vectors. EventList, e.g. calculates the histogram on the fly.
- Moved the EventWorkspace MRU list of histograms to be contained within the EventList; better code separation. Made a EventWorkspaceMRU class to encapsulate this more easily.
- Workspaces must implement getSpectrum (const/not const) to return a ISpectrum object. dataX(i), etc. methods on MatrixWorkspaces no longer need to be overridden since they call getSpectrum()
- Got rid of EventWorkspace::getEventListatPixelID() and doneLoadingData()
- replaceSpectraMap() methods offers backwards-compatibility with SpectraDetectorMaps by setting the specNo and detector list when called. Must be called AFTER creating all the spectra in a workspace.
- Refs #3318: maskWorkspaceIndex() implemented as part of ISpectrum.

Replace uses of the SpectraDetectorMap with the new interface. Note that mutableSpectraMap has now gone in favour of using MatrixWorkspace::replaceSpectraMap with a brand new implentation. Re #3067

Revert a change from some time back regarding how grouped detector positions are calculated in Absorption corrections routines. Small change in numerical result (for the better), big change in speed. Also a small refactoring in detector code that was less small in its impact on tests. Re #2276.

Tell scons to build the new TestHelpers package so that ISIS buildservers are still happy with moving the test helpers while the CMake transition is still in progress. Refs #2196 and #1881

Rename Framework code directory to just that. Also delete Main and VS & Eclipse project file. Re #1881.

Diagnostic algorithms now output a mask workspace rather than a list of spectra. Scripts have been update to handle this. Re #1903 Re #1904. Also killed a pervasive warning in MDGeometry.

Fix tests which contained references to API::Instrument. Not sure why they ran when I tried them... Re #1212

There is test that the count rate is within some number of error bars from the median be deciding if there are dead detectors.  Changes to some names and documentation refs #602 

I wrote a small helper class that reduces the amount of code needed to query detectors and access detector maps, don't know how good it is.  A new diagnostics algorithm, DetectorEfficiencyVariation, looks at white beam vanadium runs taken before and after measurements and rejects detectors whose performance changes. refs #601

New algorithm is a generalisation of FindDeadDetectors that also rejects high counting detectors.  Whether the spectra was reading high or low is recorded in the file output but currently not the output workspace refs #599

WBVMedianTest is a more descriptive name and so it replaces FindProblemDetectors detectors.  Here are some bug fixes and tests.  If this check in works I'll close the ticket re #598

When finding dead detectors it is now possible to use just some of the spectrum, e.g. to ignore an initial large pulse of neutrons to refs #597

Eliminated the variant of Workspace2D::setX/setData that takes a const reference to a vector and then copies it inside. This is inefficient since the vectors already exist once the workspace is initialized. The correct way is to take a reference to the MantidVec and assign directly to that. Re #558.