AtomicStryker's Battle Towers

AtomicStryker's Battle Towers

23M Downloads

[1.14.3] Suggestion: Add support for more structures for Finder Compass

Opened this issue ยท 6 comments

commented

Hey, I found some more structures that can be used with findNearestStructure (see https://github.com/AtomicStryker/atomicstrykers-minecraft-mods/blob/22fc6e7/FinderCompass/src/main/java/atomicstryker/findercompass/common/network/StrongholdPacket.java#L68)

Here's an enum class that lists all of the possible structures (AFAIK):

public enum StructureType {
    PILLAGER_OUTPOST("Pillager_Outpost"),
    MINESHAFT("Mineshaft"),
    MANSION("Mansion"),
    JUNGLE_PYRAMID("Jungle_Pyramid"),
    DESERT_PYRAMID("Desert_Pyramid"),
    IGLOO("Igloo"),
    SHIPWRECK("Shipwreck"),
    SWAMP_HUT("Swamp_Hut"),
    STRONGHOLD("Stronghold"),
    MONUMENT("Monument"),
    OCEAN_RUIN("Ocean_Ruin"),
    FORTRESS("Fortress"),
    ENDCITY("EndCity"),
    BURIED_TREASURE("Buried_Treasure"),
    VILLAGE("Village");

    private final String name;

    StructureType(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }
}

PS: I found these on 1.14.3, but I'm guessing they're probably also on 1.13.

PPS: I wrote the class, it's not a vanilla class. (I think) So feel free to paste it into Finder Compass.

commented

I could arguably implement a generic system instead of the Stronghold needle that accepts this ID as input instead. Question is, is searching for those structures going to be costly?

commented

Good question. My guess is that the cost will be the same if the same radius (i.e. STRONGHOLD_SEARCH_RADIUS) is used.

commented

Did not forget you asked for 1.14.3: f83f4ba

commented

1f84216 Your wish has been granted. Best delete or move your config and let it generate once, so you can see the new feature needles (one per setting).

commented

This build was pushed to curse

commented

Thank you so much!