Remove ambiguity and implement getgeom, ngeom for Wrappers.Point

[asinghvi17]

This was mentioned on Slack, the output is probably a bit better than a method ambiguity error now.

[rafaqz]

Not sure nothing is the best return value here.

We could also just return the point from getgeom and ngeom could be 1

Or error

[asinghvi17]

Error might be best?

[rafaqz]

Error is pretty safe as we can always change it later, but returning nothing is permanent

[rafaqz]

Maybe GI.getgeom(point, 1) should return the point? a bit like how numbers work just returning themselves?

Then GI.ngeom(point) should return 1

[asinghvi17]

Eh that sounds too recursive for my liking...let's just error

[rafaqz]

Where is this one at?

[asinghvi17]

Suggested change

	ngeom(::PointTrait, geom::Point) = 0
	getgeom(::PointTrait, geom::Point) = nothing
	getgeom(::PointTrait, geom::Point, i) = nothing
	ngeom(::PointTrait, geom::Point) = error("`GI.ngeom` on a Point is not allowed! To get the number of dimensions use `GI.ncoord` instead.")
	getgeom(::PointTrait, geom::Point) = error("`GI.getgeom` on a Point is not allowed! To get the number of dimensions use `GI.getcoord` instead, or `GI.x, GI.y, GI.z, GI.t`")
	getgeom(::PointTrait, geom::Point, i) = error("`GI.getgeom` on a Point is not allowed! To get the number of dimensions use `GI.getcoord` instead, or `GI.x, GI.y, GI.z, GI.t`")