branchrust_reboot/main/naval_update/sprinkler_vis_refactorcancel
5 Commits over 0 Days - ∞cph!
Remove mobile splashables from mobilePositions when deregistering
Fixed sprinklers not applying water to planters on their first tick, would have worked as normal on subsequent ticks
Tests now pass
Added a cached list of dynamic objects in range of the sprinkler, only check line of sight if they move 1m away from their last position
Will remove any cost for splashing dynamic, non-moving objects (sleepers, dud explosives, idle players, etc)
Possible refactor of how we handle finding Splashable entities for the sprinkler
Uses a new SplashableStaticGrid that supports updating grid positions of deployables that are parented to moving objects (like boats)
Use a new trigger attached to the sprinkler for dynamic entities (players, explosives, bees, etc), this actually results in much more responsive behaviour
This removes the expensive Vis.Entities but has unfortunately introduced a raycast on the dynamic entites for every splash, still needs some thought
