[suggestion]documents for get_intersection needs more detail

[MeteorShower2004]

Here is the link to official documents about get_intersection

If gpu_mode is enabled, it uses the GPU to detect the mouse. This works wherever the terrain is visible, even outside of regions, but may need to be called twice.

The document says that you can use gpu_mode=true to get the height of non-region terrain. (aka. the area outside the terrain)

However in the cpp implementation, get_intersection, actually detect the direction first and when it finds out the (direction-Vec3(1, -1, 0)).length_squared()<0.00001(source code here ), it falls back to use get_height, which can't get the height of non-region actually.

(After experimenting, I figured out a vector that could bypass that check. It's Vec3(0, -1, 0.005).)

This results in inconsistence between documents and the acutal behavior of the function.
So I suggest that the document should get fixed, or the implementation shoud match with the describtion in document.

Thanks.

[aidandavey]

I can look at this in #712

I think the implementation can be fixed - this is a work around because look_at will not work when the target vector is aligned with Vector3.UP

But since we know it's facing down (or close to), we should be able to set the rotation accordingly and skip the look_at

[MeteorShower2004]

For more details, see issue #830 and pr #712.

[TokisanGames]

This will be fixed by #840. Closing for now. Discuss in #830 or the PR.