From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: Taiyou06 Date: Wed, 19 Mar 2025 18:41:56 +0100 Subject: [PATCH] Use BFS on getSlopeDistance Uses Breadth First Search (BFS) to optimize getSlopeDistance Paper: ~75ms Leaf: ~48ms (-36%) This should help drastically on the farms that use actively changing fluids. diff --git a/net/minecraft/world/level/material/FlowingFluid.java b/net/minecraft/world/level/material/FlowingFluid.java index 4625bd55e1cb01dfb9921dcd033f05b4a8f9ad74..c0f78556d112e59333ace60f1522e7fd1efe71c3 100644 --- a/net/minecraft/world/level/material/FlowingFluid.java +++ b/net/minecraft/world/level/material/FlowingFluid.java @@ -342,32 +342,81 @@ public abstract class FlowingFluid extends Fluid { protected abstract void beforeDestroyingBlock(LevelAccessor level, BlockPos pos, BlockState state); - protected int getSlopeDistance(LevelReader level, BlockPos pos, int depth, Direction direction, BlockState state, FlowingFluid.SpreadContext spreadContext) { - int i = 1000; + // Leaf start - Use BFS on getSlopeDistance + protected int getSlopeDistance(LevelReader level, BlockPos startPos, int initialDepth, Direction excludedDirection, BlockState startState, FlowingFluid.SpreadContext spreadContext) { + it.unimi.dsi.fastutil.longs.LongSet visited = new it.unimi.dsi.fastutil.longs.LongOpenHashSet(512); + java.util.Queue queue = new java.util.ArrayDeque<>(256); + + for (Direction dir : Direction.Plane.HORIZONTAL) { + if (dir == excludedDirection) continue; + + BlockPos neighborPos = startPos.relative(dir); + BlockState neighborState = spreadContext.getBlockStateIfLoaded(neighborPos); + if (neighborState == null) continue; + + // Check if the fluid can actually pass through to this first neighbor before adding + FluidState neighborFluidState = neighborState.getFluidState(); + if (!this.canPassThrough(level, this.getFlowing(), startPos, startState, dir, neighborPos, neighborState, neighborFluidState)) { + continue; + } + long visitKey = encodeSlopeNode(neighborPos, dir.getOpposite()); + if (visited.add(visitKey)) { + queue.add(new FlowingFluid.SlopeDistanceNode(neighborPos, initialDepth, dir.getOpposite(), neighborState)); + } + } - for (Direction direction1 : Direction.Plane.HORIZONTAL) { - if (direction1 != direction) { - BlockPos blockPos = pos.relative(direction1); - BlockState blockState = spreadContext.getBlockStateIfLoaded(blockPos); // Paper - Prevent chunk loading from fluid flowing - if (blockState == null) continue; // Paper - Prevent chunk loading from fluid flowing - FluidState fluidState = blockState.getFluidState(); - if (this.canPassThrough(level, this.getFlowing(), pos, state, direction1, blockPos, blockState, fluidState)) { - if (spreadContext.isHole(blockPos)) { - return depth; - } + int slopeFindDistance = this.getSlopeFindDistance(level); + int minDistance = 1000; - if (depth < this.getSlopeFindDistance(level)) { - int slopeDistance = this.getSlopeDistance(level, blockPos, depth + 1, direction1.getOpposite(), blockState, spreadContext); - if (slopeDistance < i) { - i = slopeDistance; - } - } + // Process the queue + while (!queue.isEmpty()) { + FlowingFluid.SlopeDistanceNode current = queue.poll(); + if (spreadContext.isHole(current.pos)) { + return current.depth; + } + + if (current.depth >= slopeFindDistance) continue; + + for (Direction dir : Direction.Plane.HORIZONTAL) { + if (dir == current.excludedDir) continue; + + BlockPos nextPos = current.pos.relative(dir); + BlockState nextState = spreadContext.getBlockStateIfLoaded(nextPos); + if (nextState == null) continue; + + FluidState nextFluidState = nextState.getFluidState(); + if (!this.canPassThrough(level, this.getFlowing(), current.pos, current.state, dir, nextPos, nextState, nextFluidState)) { + continue; + } + + long visitKey = encodeSlopeNode(nextPos, dir.getOpposite()); + if (visited.add(visitKey)) { + queue.add(new FlowingFluid.SlopeDistanceNode(nextPos, current.depth + 1, dir.getOpposite(), nextState)); } } } - return i; + return minDistance; + } + + private static long encodeSlopeNode(BlockPos pos, Direction excludedDir) { + return ((long) pos.getX() & 0xFFFFFFFFL) << 32 | ((long) pos.getZ() & 0xFFFFFFFFL) << 4 | (excludedDir.ordinal() & 0x0F); + } + + private static class SlopeDistanceNode { + final BlockPos pos; + final int depth; + final Direction excludedDir; + final BlockState state; + + SlopeDistanceNode(BlockPos pos, int depth, Direction excludedDir, BlockState state) { + this.pos = pos.immutable(); + this.depth = depth; + this.excludedDir = excludedDir; + this.state = state; + } } + // Leaf end - Use BFS on getSlopeDistance boolean isWaterHole(BlockGetter level, BlockPos pos, BlockState state, BlockPos belowPos, BlockState belowState) { return canPassThroughWall(Direction.DOWN, level, pos, state, belowPos, belowState)