EnigmaticaModpacks / Enigmatica6

The official repository and issue-tracker for Enigmatica 6
https://www.curseforge.com/minecraft/modpacks/enigmatica6
241 stars 519 forks source link

Easy villagers #4574

Closed tyler489 closed 2 years ago

tyler489 commented 2 years ago

URL

https://www.curseforge.com/minecraft/mc-mods/easy-villagers

Why would you like the mod added?

I run a small server of about 10 people on at any time, on average, 40 villagers are loaded at any given time and take up anywhere from 10-85ms due to their attempt at pathfinding. Easy villagers allows you to pickup and store villagers inside the trader block, so they do not have a pathfinding qi anymore, I would love a way to keep our villagers on the server without having Massive Tps drops from villagers.

tyler489 commented 2 years ago

I saw issue #640, maybe leave only the trader and breeder blocks enabled and put them as a midgame item, say mekanism diamond alloy or something simular?

NielsPilgaard commented 2 years ago

I'm sympathetic to the issue, but I still don't want to add the mod.

Perhaps we can find a way to allow players to disable villager AI, what do you think?

TemperedFool commented 2 years ago

Would Create Seats or IF Stasis Chamber serve that function?

Discomanco commented 2 years ago

That could maybe cause issues with interacting with workstations.
A vanilla method is to put magma blocks under their workstations (assuming 1x1 holding chambers), which would block their pathfinding further than 1 block

TemperedFool commented 2 years ago

Yeah but once their profession is locked in, they don't need their workstations right?

Discomanco commented 2 years ago

They need their workstations to restock

TemperedFool commented 2 years ago

This is why I don't run a villager farm 🤣

tyler489 commented 2 years ago

I'm sympathetic to the issue, but I still don't want to add the mod.

Perhaps we can find a way to allow players to disable villager AI, what do you think?

I tried using commands to disable Ai and that also removes their ability to restock unfortunately.

MuteTiefling commented 2 years ago

Would Create Seats or IF Stasis Chamber serve that function?

I can confirm that they restock fine while seated in a Create seat, and this disables their pathfinding AI as well.

theboo commented 2 years ago

Personally recommend the server getting together and creating a public trading hall, and have them set on create seats which disables their pathfinding, but they are still able to restock with adjacent workstations

MuteTiefling commented 2 years ago

Absolutely. Servers are the best place to share resources like that... Same goes for things like Mek Fission/fusion. Why do you need 1 per player?

It's a perfect excuse to go have fun building a big building together that you can all enjoy.

ghost commented 2 years ago

If I may, there are also servers like the ones I play with my friends where the whole server is about PVP, raiding, etc.

jlfree12 commented 2 years ago

If you’re going to go the route of recommending the create seats you should add it as a loading screen tip.

tyler489 commented 2 years ago

we setup a trading hall, I was unaware create seats would allow them to still restock, unfortunately they can't breed while seated, so still going to have them moving while trying to get more

ghost commented 2 years ago

I personally think create seats should be disable because they allow you to cheese bosses. Plus Easy Villagers just allows for compact less laggy villagers. If you don't want the other stuff like iron farms just disable them. Breeders and traders are all we need since they help reduce lag a ton. It's a pretty small QoL mod

jlfree12 commented 2 years ago

can’t you also cheese bosses with minecarts/boats etc.

tyler489 commented 2 years ago

Yep, thats all I wanted, but oh well.

MuteTiefling commented 2 years ago

You're always free to add it yourself.