Dynamic tree constructor can't handle non-empty cloud

[jefesaurus]

Here's the existing constructor.

		KDTreeSingleIndexDynamicAdaptor(const int dimensionality, DatasetAdaptor& inputData, const KDTreeSingleIndexAdaptorParams& params = KDTreeSingleIndexAdaptorParams() , const size_t maximumPointCount = 1000000000U) :
			dataset(inputData), index_params(params), distance(inputData)
		{
			if (dataset.kdtree_get_point_count()) throw std::runtime_error("[nanoflann] cannot handle non empty point cloud.");
			treeCount = std::log2(maximumPointCount);
			pointCount = 0U;
			dim = dimensionality;
			treeIndex.clear();
			if (DIM > 0) dim = DIM;
			m_leaf_max_size = params.leaf_max_size;
			init();
		}

Would there be anything wrong with removing the assertion and then adding a line:
addPoints(0, dataset.kdtree_get_point_count() - 1);
right after init?
This seems more intuitive.

[jefesaurus]

Also, another question, is there a reason the input cloud's reference is non-const?

[jefesaurus]

Seems to work? #73

[jlblancoc]

👍 This one is covered by #73, so I think we can close it.
Thanks!