When generating points that lie on a sphere, the initial points generated do not appear to be truely random as values recur in subsequent points in the following pattern:
P1 = (a, b, c)
P2 = (b, c, d)
P3 = (c, d, e)
Note that this is when the point is first generated. The function later manipulates the point which obfuscates this lack of randomness.
This was discovered by reading through the code. If this project had more contributors someone else could have reviewed this and stopped this silly mistake being pushed to develop.
The cause is incorrect iteration through the range of the desired number of points, should probably iterate through the randomly generated values, incrementing by 3 each time.
When generating points that lie on a sphere, the initial points generated do not appear to be truely random as values recur in subsequent points in the following pattern:
P1 = (a, b, c) P2 = (b, c, d) P3 = (c, d, e)
Note that this is when the point is first generated. The function later manipulates the point which obfuscates this lack of randomness.
This was discovered by reading through the code. If this project had more contributors someone else could have reviewed this and stopped this silly mistake being pushed to develop.
The cause is incorrect iteration through the range of the desired number of points, should probably iterate through the randomly generated values, incrementing by 3 each time.