UI plot formatting update

Make KafkaDecoder tests run lockstep to fix races

Fix MSVC nodiscard warning from shared_ptr.get()

Change boost::dynamic_pointer_cast to std::dynamic_pointer_cast in AddPeakTest

LoadEventNexus file loader speed up by caching and reusing metadata

Redesign axes tab in figure options to accommodate both 2D and 3D plots

Restrict Peaks that Can be Added to a PeaksWorkspace to the Same Instrument

LoadWAND should add the duration log

LoadNexusLogs now extends NexusHDF5Descriptor
Allow HDF5 open to handle multiple handlers in NexusHDF5Descriptor

Expose m_fileInfo from NexusFileLoader

To do:
LoadNexusMonitors2

Add helper functions for parsing ISIS journal files

Improve extrapolation beyond range of supplied attenuation profile

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.

Replace boost::shared_ptr with std::shared_ptr

This means we can hide the parsing function that takes an xml string from the public interface and add a friend class to test this.
It also means that getRuns() can be called multiple times with different criteria without having to re-parse the file.

Re #26897

Shorten commit length requirement to 10 characters

Expose background subtraction options in GUI