These are no longer necessary as the smart pointer will clean itself up

This change will propagate outwards into everywhere they are used - need
to look at all calls to create.

As before, replaced bool arrays with boost::container::vectors on stack

Left coords alone as comment suggests it is speed-critical

Left dataSet alone as it needs to outlive the factory object, so should not
be deleted

Refs #11209

ParaView really doesn't like progress values greater than 1 and that's
what was happening. I think we didn't see this issue more frequently
due to the shear volume of progress updates being thrown at the system.

1) Fix the problem with warnings being emitted as a result of duplicate calls to connect.
2) Fix problem with ThresholdRange objects, particularly MedianAndBelow.

There is still an outstanding problem with the non-axis aligned binning plugin that I need to fix.

1) Fix the problem with warnings being emitted as a result of duplicate calls to connect.
2) Fix problem with ThresholdRange objects, particularly MedianAndBelow.

There is still an outstanding problem with the non-axis aligned binning plugin that I need to fix.

I think. Later I will replace c-style casts with c++ static_cast<coord_t>

 Also add features to cause dimensionality checks to be ignored.

Also moved NullCoordTransform to API. Implemented getLinePlot for MDEW

Refs #3907: Removed MDCell, MDPoint, Coordinate, SignalAggregate. Commented out parts of SimulateMDDTest which will need to be reworked a bit.