Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved Dragging & Entity Interactions on Ships [Pathfinding, POIs, Raids, etc.] #1027

Merged
merged 23 commits into from
Dec 27, 2024

Conversation

ThePlasticPotato
Copy link
Contributor

@ThePlasticPotato ThePlasticPotato commented Dec 15, 2024

This PR fixes the long-standing bug with extreme entity dragging delay, by allowing the client to guesstimate the mob's position.

  • Mobs are now dragged perfectly, even at ludicrous speed

  • Mob pathfinding on ships has been mildly tweaked

  • Mob rotation on ships has been fixed

  • Mob POI finding on ships has been enabled

  • Player targeting of blocks now works on ships while at high speeds or rotations.

  • Players can now hold open GUIs while on ships at high speeds or rotations.

@ThePlasticPotato ThePlasticPotato changed the title Entity Dragging Improved Dragging & Entity Interactions on Ships [Pathfinding, POIs, Raids, etc.] Dec 17, 2024
@Rubydesic
Copy link
Contributor

Can you fix the froge server failing? I think you're using some client side classes in a froge mixin

@ThePlasticPotato
Copy link
Contributor Author

Can you fix the froge server failing? I think you're using some client side classes in a froge mixin

I didn't notice that, I honestly thought those tests were just broken- I can fix it

Copy link
Contributor

@Rubydesic Rubydesic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ThePlasticPotato ThePlasticPotato merged commit 8f853da into 1.18.x/main Dec 27, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants