Refactor of TanukiEgapFinder + AutoSearch

[Mars190]

The Module EgapFinder is a much more readable and maintainable version fo TanukiEgapFinder with the option of a simple AutoSearch as well. I plan to write an AutoSearch that is not moving you on a straight line but in a spiral.

I appreciate suggestions / any help as this is my first big work on a metero-client moduel.

[cqb13]

It looks good, thank you for taking the time to clean up the code. Just a couple small things, could you please change the file name back to TanukiEgapFinder and add back the link to the original module from tanuki instead of just linking to the repository.

/**
 * Original from Tanuki:
 * https://gitlab.com/Walaryne/tanuki/-/blob/master/src/main/java/minegame159/meteorclient/modules/misc/EgapFinder.java
 */

[Mars190]

Currently I use a timer to determine when auto search should go locate the next search area, could I somehow detect through meteor if every chunk in the render distance is loaded instead of a simple 10 second cooldown? If you optimize that then you could get a few seconds out of every search area, which adds up.

[cqb13]

ChunkManager.isChunkLoaded() is probably what you are looking for.

I would try something like this

public boolean isRenderDistanceFullyLoaded() {
        var chunkManager = mc.world.getChunkManager();

        ChunkPos playerChunk = mc.player.getChunkPos();
        int renderDistance = mc.options.getViewDistance().getValue();

        int centerX = playerChunk.x;
        int centerZ = playerChunk.z;

        for (int dx = -renderDistance; dx <= renderDistance; dx++) {
            for (int dz = -renderDistance; dz <= renderDistance; dz++) {
                int chunkX = centerX + dx;
                int chunkZ = centerZ + dz;

                if (!chunkManager.isChunkLoaded(chunkX, chunkZ)) {
                    return false;
                }
            }
        }

        return true;
    }

[Mars190]

Thanks, I'll try that later, currently I'm trying out the onChunkData event with a simple tick counter

[Mars190]

This would be the pr from my side, if see where I could make some improvements, if I can I'll implement them. I tested the module on two seperate worlds and it worked good enough, sometimes there was the error Can't replace block or something like that but that was a problem beforehand as well. If you want to be sure you don't miss any chests I'd say turn down the renderDistance value to your render distance - 1 so you have overlapping chunks (though it is less efficient)

[cqb13]

all good

[Mars190]

I now just realized that debug should either be false, should be removed or should be turned into a user option instead of just being on. I can do that tomorrow.