Resolves the race between the test thread and Kafka thread when Kafka is
waiting for extractData to be called. We now use the callback mechanism
in waitForRunObservation.

Also adds some mutex's so the kernel can see when a CV should be
rechecked, atomics work because eventually the scheduler gives in and
tries again (hence we need pred.s). However, if we protect the atomic
access the other thread is resumed when the lock is released from the
original thread.

Converts the Kafka[Event/Histo]StreamDecoder tests to run in lockstep
whenever possible. The previous version held the unit test for n number
of Kafka steps then let both threads run free.

This fixes numerous races resulting in flakey tests, for example is
HistoStreamDecoder we waited for 1 iteration whereas three was required.
Rather than just tweaking the values and hoping this new mechanism will
deadlock if the threads get into a race-y condition. A timeout ensures
that after 2 minutes we fail with an abort stating we deadlocked.

This flushed out multiple races in the StreamDecoder too, however some
tests which rely on the EndOfRunStream have to run unsyncronised: The
the thread will block and the Kafka thread waits for ExtractData to be
called, causing a deadlock. If we are having problems in the future that
would be the first place to look, however for this commit it's out of
scope.

Equality for sample run

IndirectReduction GUI slowing processing on workbench when open.

NexusFileLoader class

This caused an issue with nested imports on Qt < 5.9

Fix bugs with restoring defaults on the reflectometry GUI

Fix test_fitpropertybrowser unreliable test

Remove algorithm proxy class

Improve Status Information in Eng Diff GUI

Corrected Fermi and sample distance on MAPS for PyChop

Add mechanism to print exception in Kafka test thread

cancel algorithm when cancelling script in workbench

The fitpropertybrowser was segfaulting when creating the editors within the init.
This is because the maps containing the property managers and factories are not cleared in the browser destructor as the properties can still be used elsewhere. In testing however these are not cleared and result in pointers to the managers and factories remaining even though the manager has been deleted.
This commit removes the relevant editors from these maps when the destructor is called.

Also operator!=
Operators provided for LogManager, PropertyManager, Geometry and Run alongside testing

Without these added checks the contour plot was being redrawn any time any workspace was replaced or deleted from the ADS leading to significant slowdowns in some operations when the interface was open.

Replace polaris output rebinning with delta_r and delta_q