When switching from an integrable workspace to a non-integrable one,
there was an issue with the top of the instrument widget, with was shown
as an artifact.
When switching from a non-integrable workspace to an integrable one, the
mini-plot didn't expand correctly.

When the workspace is replaced, all of the widgets in the pick tab are
reset to a void status, and the tool comes back to pixel select.

Also, if the workspace is not integrable, the plot is folded.

Make it so it is possible to draw sectors in the pick tab too.

When collapsing the integration plot in the pick tab if it is useless,
collapses it through the caption rather than hiding it, so it actually
knows it is collapsed.

Previously these were set to -1, but the test for colouring was just if value > -1.
So all negative values were coloured "invalid".

This changes the flag value to be negative double::max, and make the test equal.
This does not affect negative values (apart from -Double::max, which must be v rare).

This also uses a common constant rather than several magic -1's.

Also checks if the integrated range value is NaN (because there were no valid values, and will make that invalid).

Also removed a commented out Parrallelisation clause.  No explanation why it was commented.
I went back through blame and it only seemed to appear already commented.
Removed to save anyone else the confusion I suffered.

When deciding whether or not to show the integration bar, take into
account the possibility of EventWorkspaces or unaligned bins.

On the instrument widget pick tab, the plot which shows the spectra
for all wavelength bins for the detector is useless when the workspace
is monochromatic. As such, it is collapsed if the workspace is
monochromatic.

Giovanni Di Siena authored Mar 31, 2020 and Gigg, Martyn Anthony committed Apr 07, 2020

In places other substitutions have been made, e.g
Clang does not yet specialize std::shared_ptr for T[]. Vector
has been used instead. The operator[] methods were incorrectly
marked const but returning a non-const reference - this has been fixed.
Refs #25842

Replaced in tests and code with AlgorithmManager::create

Not expecting to pass all tests yet, just checkpointing

Also updated class_maker.py

The overwhelming use case for this method is to
take ownership of a lattice object into Sample.
Using unique_ptr clarifies that this is happening
at the call site and forces a caller to make copy
if that is truly the desired behaviour. The old code
make a copy internally but this was not obvious
from the method call.

re #27319

Prepare data for plots was expecting a detector index into an array
rather than the structured detector ID in the data file.

Previously the HKL values would only show when when viewing a single
detector. Now when plotting with a collection of detectors the HKL
values will also be plotted in the miniplot.

Previously it would update the plot when the mouse
had moved a tiny amount but the picked detector ID was
the same and wasted resources.

Project saving does not yet exist in newer builds and
will not take the same form.

Adds a Zoomer object to the mplcpp library to adds
zooming capabilities to an existing canvas. This works
differently to the Qwt4 version and has a home button
to zoom out and leaves the right-click button to
exclusively control the context menu.

Removes the requirement on the matplotlib-based widget
to implement some functionality that was only used for setup.

The data related to a single curve is now kept closer
together.

Readability is improved along with the ability to sanity
validity of the arrays given to the miniplot.

MiniPlotMpl and Colorbar classes have introduced that have
a public to match their Qwt counterparts. Nothing yet functions.

Update copyright headers in all files

Squashed commit of the following:

* First version of a script to manage copyright statements

re #23468

* Neatened up script, added comments

re #23488

* move script to tools directory

re #23488

* small script changes and a couple of manual file changes

re #23488

* Minor chnage to whitespace detection in regex

re #23488

* Add an excluded directory

re #23488

* remove a repeasted copyright statement in a file

re #23488

* Don't comsume the comment end if it is on the same line

re #23488

* fix error in new copright parsing

re #23488

* remove double copyrifght entry

re #23488

* Improve handling of old copyrights at the start of comments

re #23488

* remove empty comments re #23488

* exclude gsoapgenerated directories

re #23488

* Sort out greedy line matching re #23488

* improve empty comment removal

re #23488

* improve false positives

re #23488

* impressive speedup by limiting regex matching length

re #23488

* remove evil invisible non ascii character

Also upadte the copyright at the same time

re #23488

* resolve multiple copyrights in a single file

re #23488

* resolve an issue with new statement detection

re #23488

* another unprintable unicode character

re #23488

* pep updates and cmake the new copyright fit clang format

re #23488

* update already done new format headers

re #23488

* wrong type of bracket

re #23488

* Update class_maker and friends

re #23488

* Update all copyright statements

re #23488

* clang format re #23488

* flake8 warnings re #23488

* Flake8 warnings re #23488

* Exclude .cmake.in and rb.in files

re #23488

* replace missing line re #23488

* exclude .py.in files as they are flasely recognized as C++

re #23488

* another setp.py.in re #23488

* another .py.in correction re #23488

* Hopefully the last of the .py.in files re #23488

* resolve utf-8 encoding of python files and changed ABINS checksum

re #23488

* updates to unit tests that reference line numbers

re #23488

* remaining unit test files and other fixes

re #23488

Renames the widget to live alongside a separate implementation
that does not have a draggable scale. These will continue
to support the legacy Qt4-based widgets.

This also prepares for a matplotlib-based implementation
Refs #20240

Update to clang-format-5 and reformat the codebase 

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.