Skip to content

1.21.4#11707

Merged
kennytv merged 38 commits into
masterfrom
dev/1.21.4
Dec 5, 2024
Merged

1.21.4#11707
kennytv merged 38 commits into
masterfrom
dev/1.21.4

Conversation

@kennytv
Copy link
Copy Markdown
Member

@kennytv kennytv commented Dec 5, 2024

For tracking purposes

kennytv and others added 30 commits December 3, 2024 17:58
Upstream has released updates that appear to apply and compile correctly.
This update has not been tested by PaperMC and as with ANY update, please do your own testing

CraftBukkit Changes:
c294e05d7 SPIGOT-7975: Fix issue with Pale Sapling growing
c9f5a8fdf SPIGOT-7974: Fix Crash for Creaking Heart Block particle
VoxelShape coordiantes generally are an integer + a sum of powers of
two between [-1, -3]. Most offsets are generally an integer. As
a result, applying an offset to the coordinates generally results
in an error of 0. However, coordinate inputs do not follow such
trends. Thus, when applying an offset to the coordinate input,
there may be some floating point error.

By applying the offset to the VoxelShape coordinates, we can
eliminate additional floating point error.

This change also fixes the inconsistency when using
the single AABB, as input coordinates were not offset
when using the single AABB as the single AABB is already
offset.

Fixes Tuinity/Moonrise#81
This specific issue is caused by floating point error resulting
in the falling anvil's y position becoming around -8E-17 when it
should be 0.
While this is still very comfortably in the collision
epsilon (1.0E-7), this results in the falling anvil's y block
position to become -1 (as the block position is simply
the floor of the coordinate).
setup-gradle v4 validates the wrapper
@kennytv kennytv merged commit 4c39ea2 into master Dec 5, 2024
@kennytv kennytv deleted the dev/1.21.4 branch December 5, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

6 participants