improve stability of minimap

(though full refactor is still needed)

Author: zardoy

renderer/viewer/lib/mesher/mesher.ts

@@ -2,6 +2,7 @@ import { Vec3 } from 'vec3'
 import { World } from './world'
 import { getSectionGeometry, setBlockStatesData as setMesherData } from './models'
 import { BlockStateModelInfo } from './shared'
+import { INVISIBLE_BLOCKS } from './worldConstants'
 
 globalThis.structuredClone ??= (value) => JSON.parse(JSON.stringify(value))
 
@@ -156,10 +157,11 @@ const handleMessage = data => {
         for (let x = 0; x < 16; x++) {
           const blockX = x + data.x
           const blockZ = z + data.z
-          blockPos.x = blockX; blockPos.z = blockZ
-          blockPos.y = 256
+          blockPos.x = blockX
+          blockPos.z = blockZ
+          blockPos.y = world.config.worldMaxY
           let block = world.getBlock(blockPos)
-          while (block?.name.includes('air')) {
+          while (block && INVISIBLE_BLOCKS.has(block.name)) {
             blockPos.y -= 1
             block = world.getBlock(blockPos)
           }

renderer/viewer/lib/mesher/shared.ts

@@ -3,6 +3,8 @@ import { BlockType } from '../../../playground/shared'
 // only here for easier testing
 export const defaultMesherConfig = {
   version: '',
+  worldMaxY: 256,
+  worldMinY: 0,
   enableLighting: true,
   skyLight: 15,
   smoothLighting: true,

renderer/viewer/lib/worldrendererCommon.ts

@@ -560,7 +560,9 @@ export abstract class WorldRendererCommon<WorkerSend = any, WorkerReceive = any>
       textureSize: this.resourcesManager.currentResources!.blocksAtlasParser.atlas.latest.width,
       debugModelVariant: undefined,
       clipWorldBelowY: this.worldRendererConfig.clipWorldBelowY,
-      disableSignsMapsSupport: !this.worldRendererConfig.extraBlockRenderers
+      disableSignsMapsSupport: !this.worldRendererConfig.extraBlockRenderers,
+      worldMinY: this.worldMinYRender,
+      worldMaxY: this.worldMinYRender + this.worldSizeParams.worldHeight,
     }
   }
 

src/react/MinimapProvider.tsx

@@ -45,7 +45,7 @@ export class DrawerAdapterImpl extends TypedEventEmitter<MapUpdates> implements
   chunksStore = new Map<string, undefined | null | 'requested' | ChunkInfo>()
   loadingChunksQueue = new Set<string>()
   loadChunk: (key: string) => Promise<void> = this.loadChunkMinimap
-  mapDrawer = new MinimapDrawer(this.loadChunk, this.warps, this.loadingChunksQueue, this.chunksStore)
+  mapDrawer = new MinimapDrawer(this.loadChunk.bind(this), this.warps, this.loadingChunksQueue, this.chunksStore)
   currChunk: PCChunk | undefined
   currChunkPos: { x: number, z: number } = { x: 0, z: 0 }
   isOldVersion: boolean