PQS.sphere.sx structure is non-linear about kspcommunityfixes HOT 3 CLOSED

So, to be clear, we are talking about PQSLandControl.OnVertexBuildHeight() ?

This piece of code is indeed quite hard to follow. This pattern of having instance fields being used as local variables shared by multiple methods all over the place is really awful.

Despite this being quite weird, I think sx is designed to be in the [-0.25, 0.75] range. There are way too many places where this is used for this to be a mistake. Since this is mapping to texture UV coordinates, my interpretation is that there is a deliberate intent to have a 90° offset between the texture origin and the 0° longitude.

And in fact, when using those values as UV coordinates, the stock code always does val = Math.Abs(val - Math.Floor(val)) which is equivalent as your modified code to transform back the value to a valid [0, 1] UV range.

I think the actual mistake is indeed PQSLandControl.OnVertexBuildHeight() trying to get back a longitude value from the sx coordinates without accounting for that offset, so your fix makes sense from that PoV.

What I'm unsure of is how this will interact with the "intended" scatter placement. Fixing this like that mean whatever stuff is used to configure the distribution is working with a "the origin is at 90° longitude" assumption, but I guess that's already accounted for by people using this. Still, a confirmation that scatters are actually where they are supposed to be would be nice.

I will implement that fix and try to do a release ASAP.

On what this fix should be described, what is PQSLandControl used for ? Does it only affect scatters distribution or is the class used for other purposes ?

from kspcommunityfixes.

Yes you got the right function. And looking at it I agree with everything you say... just seems very odd a choice of coordinate system.

Will test soon.

from kspcommunityfixes.

Forgot to answer one point: LandControl can do a bit more than scatters. It can do some terrain coloring effects amongst other things. Mainly scatters though.

from kspcommunityfixes.