giers10/webeditor3d
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move boundary sample skipping logic to lowered edge vertex collection

Browse Source
This commit is contained in:
Victor Giers
2026-05-21 07:59:03 +02:00
parent a60a963a00
commit ee65a479e3
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
tests/geometry/terrain-mesh.test.ts
View File

@@ -56,19 +56,11 @@ function collectTopEdgeKeys(options: {
options.level.positions[positionOffset + varyingOffset]! / options.level.positions[positionOffset + varyingOffset]! /
options.terrain.cellSize; options.terrain.cellSize;
const roundedVaryingSample = Math.round(varyingSample); const roundedVaryingSample = Math.round(varyingSample);
const maxVaryingSample =
options.axis === "x"
? options.terrain.sampleCountZ - 1
: options.terrain.sampleCountX - 1;
if (Math.abs(varyingSample - roundedVaryingSample) > epsilon) { if (Math.abs(varyingSample - roundedVaryingSample) > epsilon) {
continue; continue;
} }
if (roundedVaryingSample === 0 || roundedVaryingSample === maxVaryingSample) {
continue;
}
const sampleX = const sampleX =
options.axis === "x" options.axis === "x"
? options.sampleCoordinate ? options.sampleCoordinate
@@ -122,11 +114,19 @@ function collectLoweredEdgeVertexCount(options: {
options.level.positions[positionOffset + varyingOffset]! / options.level.positions[positionOffset + varyingOffset]! /
options.terrain.cellSize; options.terrain.cellSize;
const roundedVaryingSample = Math.round(varyingSample); const roundedVaryingSample = Math.round(varyingSample);
const maxVaryingSample =
options.axis === "x"
? options.terrain.sampleCountZ - 1
: options.terrain.sampleCountX - 1;
if (Math.abs(varyingSample - roundedVaryingSample) > epsilon) { if (Math.abs(varyingSample - roundedVaryingSample) > epsilon) {
continue; continue;
} }
if (roundedVaryingSample === 0 || roundedVaryingSample === maxVaryingSample) {
continue;
}
const sampleX = const sampleX =
options.axis === "x" options.axis === "x"
? options.sampleCoordinate ? options.sampleCoordinate