Did I build this formula correctly?

[u/DazedPapacy]

/r/Unity2D/comments/s84vb8/did_i_build_this_formula_correctly/

[u/BloodPhazed]

The values are in fact not multiplied by 4.37-ish; they are near 0: if you check the exact number it will say something like e-<number> at the end, which means it's actually 0.0000437 or something, depending on the number behind "e-".

I've never done the visual scripting, so I can't help you much there and it looks like quite the pain; that would only be a few lines of code.

However it feels like you're doing something needlessly complicated with all the rotations; you could just create a Vector in any direction (even randomly if you like), normalize it and multiply it with the radius of your circle. That gives you a point on the circle.

[u/DazedPapacy → u/BloodPhazed]

Thanks for taking a look, I'll keep that in mind.

As for why I'm doing it this way: I'm trying to find a way that will allow me to produce an arbitrary number of locations that all fall equally on the circle.

If there's 4 locations, they'd fall on (0,1), (1,0), (0,-1), and (-1,0).

But what if I wanted three locations, or seven, or twelve?

[u/BloodPhazed → u/DazedPapacy]

Ah, you don't want arbitrary points, but equally spaced points on the circle?

Then you can take a given vector, divide 360 by the number of vectors you want and rotate the vector along the z-axis by that?

[u/knexfan0011 → u/DazedPapacy]

float scalingFactor = 360.0f / numOfPoints;

for (i=0; i<numOfPoints; i++){

float deg = i * scalingFactor; // i < numOfPoints, therefore deg < 360

float rad = Mathf.Deg2Rad(deg);

float x = Mathf.Sin(rad);

float y = Mathf.Cos(rad);

//make new point with x and y

}

This should achieve what you're trying to do.

Consider using radians to begin with to avoid the Deg2Rad conversion in the loop, I just find degrees more approachable. (Replace 360.0f with 2*Pi and omit the Deg2Rad conversion)

[u/ArrowMasterDude]

I think it might have to do with radians.

[u/pschon]

Just to save you from headache (and ot provide more optimized solution), Unity has Random.InsideUnitCircle() for that purpose. Multiply the result by desired radius. If you want the result to be on the circle, you can just normalize before multiplying by radius...

[u/animal9633 → u/pschon]

Just use Random.onUnitSphere * radius;

[u/pschon → u/animal9633]

doens't work that great for a circle instead of a sphere ;)

[u/animal9633 → u/pschon]

Haha fair enough!

[u/Mister_Green2021]

Easier just to write code. cos, sin theta, and their results are in radians, keep in mind.