Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

I also considered doing likewise in CSGObject,
but found this would be complicated by the use of
shapes returned by ComponentCtreationHeliper.
I did make use of auto to make such a change easier.

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Refs #21378

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

The files are yet to be renamed

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Re #19154

Re #19154

The Monte Carlo method now has two methods: one runs the simulation a
fixed number of iterations, the other manages the first and checks for
convergence.

Re #19154

Monte Carlo is used as the general (fallback)s method. Also, code for
the triangular mesh case is included, but unused. The way we create
surfaces now doesn't produce consistent surface normal directions. This
makes the triangular method fail.

Re #19154

The test now includes negative Z coordinates as well.

It will give clarity in the test and also fail in release mode.

Avoid calls to exit that could cause the program to just stop without
any warning.

Refs #16044

Assign the correct rule number when passing a single rule to
Object::procPair. The method detected that only one rule was present
correctly but failed to tell the external code which one it had found.
This led to the rule map being updated incorrectly.

The commit also switches the order in which the rules are assigned in
Intersection constructor to be consistent with the rest of the class
and the Union operation.