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

Pathfinding on 1.13 not "dynamic" but rather "checkpointy" #1880

Open
MiniDigger opened this issue Jul 24, 2019 · 1 comment

Comments

@MiniDigger
Copy link
Contributor

commented Jul 24, 2019

This is kinda hard to explain, its better seen in the video down below.
Basically the NPC doesn't seem to track me dynamically but checks like once a second, moves to that location and then tries to attack me.
I talked to @mcmonkey4eva in discord trying to debug this, enabling use-new-finder: true didn't fix this, we also can't reproduce it on 1.14.
He said that this seems to be a general issue with citizens pathfinding, thats why I am reporting it here as opposed to the sentinel tracker.

Video: https://streamable.com/tr9zz
Video with path debug: https://streamable.com/t7jj6

Versions:

version
[12:21:44 INFO]: This server is running Paper version git-Paper-641 (MC: 1.13.2) (Implementing API version 1.13.2-R0.1-SNAPSHOT)
[12:21:44 INFO]: Previous version: git-DyescapePaper-154ec17 (MC: 1.13.2)
[12:21:44 INFO]: You are running the latest version
version citizens
[12:22:02 INFO]: Citizens version 2.0.25-SNAPSHOT (build 1737)
[12:22:02 INFO]: Website: http://www.citizensnpcs.co
[12:22:02 INFO]: Author: fullwall
version sentinel
[12:22:06 INFO]: Sentinel version 1.9.2 (build 261)
[12:22:06 INFO]: Author: mcmonkey

Config: https://hasteb.in/axiyugud.yaml
(Sentinel Config: https://hasteb.in/otevokoc.yaml )

@mcmonkey4eva

This comment has been minimized.

Copy link
Member

commented Jul 24, 2019

Specific to the call: getNPC().getNavigator().setTarget(entity, false);, and further specific to 1.13.2 servers.

It's behaving like it has its target set to the player's location every update cycle, rather than continually tracking towards the player (which it seems to more properly achieve on 1.14.4).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.