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.

Determine the total number of detectors used in the spectrum info

Move FileDescriptor and NexusDescriptor includes to derived classes

Avoid extra multiply in Plus and Minus

Added Mantid::NeXus::NexusHDF5Descriptor class caching metadata

Populates Metadata entries in constructor
Internal reference accessible in public function
Added unit test
Address CI issue with POSIX strdup function

Improve Outputs When Focusing in Eng Diff 2 GUI

Change Algorithm State tracking

Co-Authored-By: StephenSmith25 <56431339+StephenSmith25@users.noreply.github.com>

Signed-off-by: Steven Hahn <hahnse@ornl.gov>

Stop for loop in Rebin2D asking for next element after the last bin

Remove extra indents that were causing Flake8 Jenkins failure

Makes clang-format parallel where possible

Update google test / span

Make python targets runnable on MSVC IDE

Refactored Track to use boost::container::small_vector with optimised vector size

Signed-off-by: Steven Hahn <hahnse@ornl.gov>

Signed-off-by: Steven Hahn <hahnse@ornl.gov>

Update copyright to include CSNS

Updates the CMake configuration to set our debugging targets. This means
a user can debug Workbench or System tests, which have a python entry
point, directly from the MSVC IDE.

# Conflicts:
#	Testing/SystemTests/tests/analysis/ErrorReporterServerTest.py

Re-add commented out python 3 test code

Eventlist: reserve capacity of the specified event type

Remove ErrorReporterServerTest

Needed to add checks around makedirs(), throws an error if the directory
already exists.

Also updated class_maker.py

Sets clang-format to run in parallel on build servers where the parallel
package exists