Fix & Adding Auto Check / Instant Kill#74
Fix & Adding Auto Check / Instant Kill#74K0bus wants to merge 6 commits intoploppyperson:masterfrom K0bus:master
Conversation
|
"Instant Kill" can already be achieved with the current config system - thats the entire point of the priority system. I have made an example of this: https://gist.github.com/Nathat23/2fd312df702424401679c40810feb0a3 The "Auto Check" seems incredibly wasteful. What's the point of duplicating the stacking logic - why not just set the stack value to one and let the stacking task do its job? It would also be more efficient to do this on chunk load rather than as a task. And there are also other considerations to be made, some mobs might not supposed to be stacked, which is why the current implementation is a command, '/sm forcestack' The fix for the max health error is the obvious one, however there may be a different reasons for the health disparity between the new + old entity, which should be investigated. It is important to think carefully about the effects of your code and consider why things are the way they are. I appreciate the attempt and effort though. |
Little update for fixing error when kill Pillager or any entities have more than 20 HP.
Also adding two features :