Closed idoash4 closed 9 years ago
Can one of the admins verify this patch? Type 'ok to test' to test.
Can one of the admins verify this patch? Type 'ok to test' to test.
so no merge yet? Do you want me to test it?
No it's fine you can throw this into civcraft.
ok to test
so then what does this do exactly right now?
Just add more mobs that MusterCull will attempt to merge(it was only working on pigmen so far). I didn't change anything in the code, only edited the config file.
Do we have explicit confirmation that merging is working as expected on Civcraft?
I haven't tried AFK'ing my gold farm in a week, so I can't attest one way or the other.
there seemed to be some confirmation in the testing thread on discourse, but I am not sure how explicit that was
Yes it does work as expected. Me and Max tested it on civcraft. The drops from portal farms are still low tho. Mostly because of the change to mob AI - The pigmen get stuck on the portals and it takes a long time until they die. I think the best soulation will be spawning the pigmen a block away from the portals.
I have re-enabled mob AI for tomorrow morning, you want this merged and online tomorrow morning?
Yes please. I hope the mob AI isn't going to drop the tps significantly.
@ttk2 On my explorations I figured out how to manually turn off and on mob ai per entity. If you have any ideas for that I'm all ears.
lets see how it works out with stuff turned on, anyways merging.
this is online now.
I added more mobs. I am still trying to figure out a bug with animals so they are not on this config.