Rabbit Pathfinding Fixed
Fixes rabbit pathfinding efficiently
Rabbit Pathfinding Fixed
Fixes rabbit pathfinding efficiently.
As of 1.21.1 there are multiple problems with rabbit pathfinding:
Basic pathfinding
1. The calculation of the jump height/velocity is incorrect and poorly implemented.<br/>This results in too small jumps for climbing over a block. 2. Rabbits "stall" (no horizontal movement) during jumps - due to this they just jump upwards in the same place when trying to climb a block.<br/>This behavior is caused by ``RabbitMoveControl`` which only sets the (horizontal) speed correctly during movement but not while jumping. 3. Rabbits are stuck / try to wander around forever. * The root cause is that ``EntityNavigation`` sets its timeouts based on movement speed.<br/>If the movement speed is 0 (this is the case when a rabbit/mob is "stuck"), the timeout is also 0... and if the timeout is 0 it's ignored and therefore it's executed forever (or until interrupted by something external like another goal). * Rabbits only have a single goal when idle: ``WanderAround(Far)``. Most other entities also use ``LookAroundGoal``.<br/> Thus the above mentioned infinite navigation is never stopped in favor of executing another goal like in most other mobs. * ``RabbitMoveControl#tick`` constantly updates the rabbits speed (``RabbitEntity#setSpeed``).<br/> While doing this it also indirectly executes ``moveControl#moveTo`` thus the rabbit always tries to reach it's last target even when it shouldn't do that.
> As of ``1.21.4/24w46a`` MC-150224 was fixed, correcting the jump height (1) and fixing the stall (2).<br/> > However all other parts - including optimizations and sanity checks in above mentioned fixes - are still missing.
Eating carrot crops
1. Rabbits can't reach the crops and always stop one block short of them.<br/>This is due to selecting the incorrect distance from the crop block (it's ``1`` but should be ``0``). 2. Rabbits eat the crop instantly even while still jumping and being in the air. 3. The goal/behavior is immediately aborted (after a few ticks - when a target crop block was selected) due to incorrect implementation of ``shouldContinue`` and ``isTargetPos`` methods.
Other fixes
* The client-side rotation of Adults rabbit's head is most of the time slightly upwards - MC-306682 affects 26.1 or later
Detailed video comparisons are also available.