New maxEntityCollisions optimization might break entity cramming at low values
skerit opened this issue ยท 3 comments
Ever since #1038 was implemented, entity cramming no longer works if I set maxEntityCollisions
lower than 15.
I originally set the value quite low, to 3, because then it seemed to work well enough. Entities were still quite pushable, and gold farms worked as expected.
But after updating to the latest Carpet version, I got reports from players that entity cramming was disabled.
I tested it, and indeed, after raising the cramming from 3 to 15 entities started taking damage again.
Now I see that the #1038 pull request actually fixed certain things. Is it safe to say setting the maxEntityCollisions
to 15 will provide as-good performance improvements as the previous version had at only 3?
Important to add: my world had maxEntityCramming
set to 16. If I set it to the default vanilla-value of 24, I actually have to set maxEntityCollisions
to at least 23, so it seems like you can't set the maxEntityCollisions
lower than maxEntityCramming
I will fix it, but it will mean that optimization-wise the benefits will be depending on both maxEntityCollisions as well as maxEntityCramming, whichever is higher.