refs #8125. Handle cycles for each instrument.

Allow switching between cycles for a single instrument. Display cycles to choose from. Update cycles when instrument is changed. Default to latest cycle and load run names.

Also set it up so that it extracts the run number from the workspaces as part of the transfer process so that only run number rather than workspace names appear in the table.
refs #8125. Hide and show the cycles widget as appropriate.
Lowered version 1 confidence

I've lowered teh defualt confidence of LoadAscii version 1 as it was preventing LoadAscii version 2 from being chosen by the Load selector

Refs #7732
Altered LoadAscii Cumstom dialog

The LoadAscii Dialog now changes depending on the version being invoked.

Version 2 has the new comment indicator feild

Refs #7732
Tided code and formatted

Removed some commented out code and some trailing spaces

Performed a "format Document" command form Visual studio to add indentation.

Refs #7732
refs #8125. Fix failing test.

Statement moved outside of try-except block
Added new format, removed debug code

Added the new Scientific Notation option to the dialog

Removed the std::cerr calls I added when debugging

Changed the placeing of the header code so that the header is no longer per specta, but per file

Refs #7732
Search by ID instead of name. Refs #8126.
Written the unit test for version 2

Createa s unit test for the new format based on the old test.

Compared to the original the unit test is now split into two.

Refs #7732
New version of LaodAscii written

The new verison of LoadAscii was going to be based on the original, and it has been, but not as much as was initially intended, and it ended up a full rewrite.

The Loader will load the form of files saved by SaveAscii version 2, but not version 1. The format has changed that much.

It is slgihtly flexible in that it will load files that SaveAscii wouldn't wirte, but htis only extends as far as it'll accept blank lines as equivelent to spectra IDs, will allow 2 collumns minimum (implying that E and Dx are 0), and will ingnore lines staritng wiht a comment character. However, it is still strict enough that it will expect any standards set at the start of the file to be continued thoughout. This means number of bins per spectra, number of columns per bin, inclusion or omittance of spectra IDs, comment characters and delimiters.

Refs #7732
Refs #7965 Fix broken system tests on develop
Removed 1 unused property. Refs #8101.
Add external download attribute to `Facilities.xml`. Refs #8126.
Added new attributes to cataloginfo. Refs #8126.
Merge branch 'master' into feature/7637_improve_icat_gui. Refs #7637.

Conflicts:
	Code/Mantid/Framework/ICat/src/CatalogDownloadDataFiles.cpp
	Code/Mantid/Framework/ICat/src/CatalogSearch.cpp
re #8156: Does not try to find file path twice

The _data_file is already a full_path in this stage. It is useless, and,
as showed, error_prone to try to do it.
re #8156 correct getFilePathFromWorkspace

It could be different, but it is necessary to wait to fix #8158.
Refs #7965 Change var name to match version on develop
Refs #8146 changing method of scaling output workspace.
Merge remote-tracking branch 'origin/master' into feature/7972_copy_logs_alg

Conflicts:
	Code/Mantid/Framework/Algorithms/CMakeLists.txt
Refs #7972