Improve tooltips on the reflectometry GUI

inplace specialization for SetUncertainties

Remove completed Algorithms from AlgorithmManager managed algorithms

a new report for scraping the git log

Add check to handle delete observer

Remove Bash 4.2 feature breaking Mac OSX

Removes a check that uses the -v flag, this was introduced in BASH 4.2,
so apparently breaks certain Mac builders

Previously this tooltip was misleading because it was referencing the transmission stitch params property of the reduction algorithm. This input is actually for all properties for the Stitch1DMany algorithm so we cannot take the documentation from a single algorithm property as we do for the other tooltips. Therefore, this commit adds a function to be able to set a tooltip independently of any algorithm and sets this tooltip manually.

Fix cpp check 1.90 warnings

as there is some strange interplay between datasearch.directories and defaultsave

Change output ws names in Polaris.create_total_scattering_pdf

Emit Kafka exception msg when err callback used

Add quotes to Span patching step, in case there are spaces

Haven't changed AlgorithmProxy as that class will be deleted in the next PR

Update sanitizers to use compiler instrumentation

Without this check every plot was being dedrawn every time a workspace was deleted from the ADS leading to slowdowns.

This function returns a bool which was previously being used to indicate two separate things. The return values could mean:
  * False: Either the workspace does not exist on the axis or it does but the axis is not empty after its' removal.
  * True: Workspace exists and was removed and axis is now empty

This made the value not very useful for determining whether a call to remove_workspace_artists had removed anything. A grep of the code base returned that this return value was not currently being used anywhere except in MantidAxes.cpp where it was assumed to indicate whether anything had been removed. As you can easily check whether an axis is empty by other means I have changed the return value so that it indicates whether the call resulted in any change to the plot which is harder to divine by other means and brings this function in line with replace_workspace_artist.

This commit removes a print statement initial added for debugging that was missed on cleaning up before pushing.

and fix a unit test for not having a list limit

Determine the total number of detectors used in the spectrum info

Merge branch 'master' of https://github.com/mantidproject/mantid into 28093_only_retain_running_algorithms_3

Runs clang-tidy pass by ref across the codebase to fix-up various
warnings we are seeing in cppcheck

Hand-fixes some cppcheck warnings, this was work prior to fixing the
cppcheck target.

Updates the cppcheck target to run like a typical compilation. This
means CMake will fix-up the headers required on the fly. This reduces
the time compared to adding all headers, and improves the accuracy
significantly.

Adds clangd to the gitignore, this is generated by using the language
server on a source dir in VS code

Fixes sanitizers not switching off if USE_SANITIZER was set to off.
Additionally gets rid of a space between LSAN_OPTIONS and the actual
options, which was causing the suppressions file to be ignored.

Previously Kafka dropped the exception and just incremented using the
same callback our wait. This will now print the exception message
associated, so we can distinguish between a successful and failed
callback in unit test logs.

I could have made it so that the exception is thrown in the main thread
by taking a s_ptr. However it quickly becomes unweildly for a simple
probe, and we only gain the stack trace - which was already lost when
extractData re throws the exception.

Move FileDescriptor and NexusDescriptor includes to derived classes