Agent's located outside of env bounds read from some bins twice during spatial partitioning. about flamegpu HOT 4 CLOSED

Needs to have assertion errors to stop users from allowing agents to create messages outside of the bounds of the message environment. Will be integrated in version 2.

from flamegpu.

Snip
Neither XLST or C preprocessor macro's support (clean) floating point arithmetic, so it isn't easily feasible to check this is correct at compile time (I just hacked together a quick solution using pre-processor integer arithmetic, before noticing floating point values have been used within the examples).

Presumably such a check would therefore need to be included within the initialisation function of current FLAMEGPU, this is less desirable, but I don't see a solution for warning about this earlier. It might be possible to achieve this with XLSTs limited mathematical operations, however I'm unsure whether FLAMEGPUs packaged xlst preprocessor supports these and where I can find clear documentation on their usage.

from flamegpu.

Managed to get XLST math to work correctly with alot of trial and error (removing the // or space before - seemingly breaks this code), will test examples still build and commit.

<!--Compile time error if partitioning radius is not a factor of the partitioning dimensions as this causes partioning to execute incorrectly-->
<xsl:for-each select="gpu:xmodel/xmml:messages/gpu:message/gpu:partitioningSpatial">
<xsl:if test="(//gpu:xmax - //gpu:xmin) != (floor((//gpu:xmax - //gpu:xmin ) div //gpu:radius ) * //gpu:radius)">
#error "Partitioning radius must be a factor of partitioning dimensions.\nRadius: <xsl:value-of select="gpu:radius"/>, Xmin: <xsl:value-of select="gpu:xmin"/>, Xmax: <xsl:value-of select="gpu:xmax"/>"
</xsl:if><xsl:if test="(//gpu:ymax - //gpu:ymin) != (floor((//gpu:ymax - //gpu:ymin ) div //gpu:radius ) * //gpu:radius)">
#error "Partitioning radius must be a factor of partitioning dimensions.\nRadius: <xsl:value-of select="gpu:radius"/>, Ymin: <xsl:value-of select="gpu:ymin"/>, Ymax: <xsl:value-of select="gpu:ymax"/>"
</xsl:if><xsl:if test="(//gpu:zmax - //gpu:zmin) != (floor((//gpu:zmax - //gpu:zmin ) div //gpu:radius ) * //gpu:radius)">
#error "Partitioning radius must be a factor of partitioning dimensions.\nRadius: <xsl:value-of select="gpu:radius"/>, Zmin: <xsl:value-of select="gpu:zmin"/>, Zmax: <xsl:value-of select="gpu:zmax"/>"
</xsl:if>
</xsl:for-each>

from flamegpu.

Just updated the fix for this (and overwrote the prev fix). Discussion with Paul clarified that // notation searches from the root, meaning the above fix could work incorrectly where multiple spatial partitions exist.

I tested it again without the slashes, and the earlier error seems to have disappeared, so presumably I fixed whatever was causing that (potentially incorrect division, although that would mean it doesn't parse L2R) whilst arranging the rest of the algorithm.

82d7dc9

from flamegpu.