minRegionArea and regionMergeSize ignored in pcToTileCache

[albertvanveen]

When using pcToTileCache to generate a tiled navmesh, the minRegionArea and regionMergeSize options appear to be ignored.

const { navMesh, tileCache } = pcToTileCache(meshes, {
    tileSize: 16,
    cs: 0.1,
    ch: 0.1,
    walkableRadius: 0.5,
    walkableHeight: 1,
    walkableClimb: 0.5,
    minRegionArea: 10,       // ignored
    regionMergeSize: 1,      // ignored
    tileCacheMeshProcess,
});

This appears to be due to generateTileCache applying internal defaults (minRegionSize = 8, mergeRegionSize = 20) for each tile, which overwrites the options passed to pcToTileCache.

[isaac-mason]

Hi, it looks like this could be related to another issue: #313

The recast library uses this common rcConfig struct for all it's navmesh generation approaches but not all parameters are used across all of them.

I had omitted already minRegionArea from the TileCacheGeneratorConfig type for this reason, but I'll have to check.

[isaac-mason]

It doesn't appear a regionMergeSize parameter is used by dtBuildTileCacheRegions, so we should remove the option from the TileCacheGeneratorConfig type.

https://github.com/isaac-mason/recastnavigation/blob/599fd0f023181c0a484df2a18cf1d75a3553852e/DetourTileCache/Source/DetourTileCacheBuilder.cpp#L186

[albertvanveen]

Thanks for your reply!

I just noticed that I used different naming: minRegionArea vs minRegionSize and regionMergeSize vs mergeRegionSize, so I need to check which ones are correct (at least for pcToSoloNavMesh). It seems I used wrong naming in the first place.