Apply ties in correct order

Re #22925

Fix workspace property handling in MaskDetectors

Use local static variable for singleton instance

See inline code comments for more details regarding the reasoning

    Avoids a bug in VS2017 where the call_once flag is not zero
    initialized but dynamically initialized despite its constructor
    being marked constexpr. This leaves the singleton
    initialization open to intialization ordering problems.

    A local static variables avoids the ordering problem and are
    guarateed to be threadsafe in C++ 11.

Indirect - Fix the default detector grouping option

New algorithm to slice event workspaces for ISIS reflectometry

Disable Plot Result for MSD and F(Q) Fit for one spectra

Modified Systemtests to show more failures

Move V3D operators to be inline

It was only writing the full log to console if the test status contained 'fail'
which would apply to 'failed' and 'algorithm failure' but it was missing statuses
like 'hung'

Now the output is always printed unless the status is 'success' or 'skipped'

Project Recovery: Handle multiple instances of mantid properly

Force progress messages to be str

Add new algorithm MatchSpectra

Mantidplot/ Testing/ buildconfig/ misc. typos

The problem was that empty progress messages were being converted to
`None` which `AlgorithmObserver` did not have the proper overloaded
methods to convert to an empty string. The fix is to convert them to a
string before calling the binding. For extra measure, the progress
percentage is converted to a float as well.

Refs #23632

Refs #21800