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

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

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

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.

Add feature where an externally generated attenuation profile can be loaded into
the material definition from a .DAT text file. This will override the normal
exp(-alpha*thickness) calculation of the attenuation. The new feature was requested
as part of the absorption calculation for the Pearl instrument where some parts
of the environment are crystalline

Found the lookup\interpolation on the loaded profile was slowing down the absorption
Monte Carlo calculation.
So improved performance of Kernel::Interpolation. Changed it to use vector of pairs
instead of two separate vectors to reduce amount of navigation around vectors

Considered adding feature into AttenuationProfile where I add any interpolated points
into the override attenuation profile to prevent interpolation next iteration of
Monte Carlo simulation. However wasn't convinced this was going to be thread-safe
during the parallel part of the absorption calculation - may revisit this. While doing
this investigation I fixed the following issues with the function Interpolation::AddPoint:

a) it doesn't add the same point twice
b) it reuses the lower_bound function rather than stepping through the vector sequentially

I removed the exception from findIndexOfNextLargerValue because the end()
iterator is a valid result if the x value supplied equals the last x value
already in the vector. Could have left check in for begin() but didn't seem
much point

Finally - the interpolation outside the range is not v good. The data at the extremes
is noisy so we end up extrapolating based on random gradient in extreme 2 points. Need
to either fit or use more points to determine gradient

Also updated class_maker.py

This commit replaces all header guards in Kernel with #pragma once

Bumps the version of GTest to 1.10.0, removes the previous patching
steps, which are no longer needed. Removes the extra handling which is
provided in upstreams CMake, such as managing include directories

Git only converts CRLF to LF and not CR so to produce
the same SHA as git we must do the same.

A new boolean parameter SimulateTracksForEachWavelength has been added to
the MonteCarloAbsorption algorithm. This exposes a switch on the MCAbsorptionStrategy
that was present before this change but was hardcoded to False.

The parameter NumberOfWavelengthPoints is only required if SimulateTracksForEachWavelength
is set to True. This change means that an attenuation factor is calculated for all wavelength
bins if SimulateTracksForEachWavelength=false. To facilitate this a performance enhancement
has been made to the Material class to calculate the total scatter cross section and the
absorption cross section once when the Material rather than every time these cross sections
are retrieved by the MonteCarloAbsorption calculation.

This change required some unit and system tests to be updated eg ILLDirectGeometryReductionTest
has been changed to remove the NumberOfWavelengthPoints parameter and the reference file to be
updated. To make this test more robust I have increased the number of scenarios used in the MC
simulation from 5000 to 20000

This commit adds tests to unit and PDFFourierTransform todo with the new atomic distance unit.

re: #27769

Newer boost bind seems to be causing
compiler warnings on Windows. It's no
longer required now bind is in the STL.
Refs #16578

Updates the associated instrument to one from the facility when a
facility is changed. This could fix numerous spurrious failures where
unit tests will assume that a change in facility -> a change to an
instrument from said facility

re #27458

Intersting as the last set of builds passed

re #27319

clang format - not sure why this isnt working

clang-format

clang-format

Main change is to add an overload of registerFeatureUsage that accepts a
simple string in addition to the existing method that takes a vector of strings

Update some further calls to registerFeatureUsage that had appeared in master
Also change trimming behaviour so trimming of feature names is done on copy
rather than in place

Now accepts:
- an enum for feature type rather than a string eg Algorithm
- a vector of strings for the feature name instead of string (eg class1->method1)

Also updated pre-existing calls to registerFeatureUsage in C++ and Python

- Remove bind1st/bind2nd usage.
- Avoids an internal compiler error in gcc with getProperty.
- Adds wrapper around sip executable to process the output and
  remove the throw specifiers. They are not allowed in C++ 17.
Refs #26261

It was unused and gave the impression that a wavelength
dependent calculation was being used.
Refs #25957

Refs #25957