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

Sentinel NPCs chasing a target seem to pause when close #335

Closed
Ferocimo opened this issue May 1, 2020 · 6 comments
Closed

Sentinel NPCs chasing a target seem to pause when close #335

Ferocimo opened this issue May 1, 2020 · 6 comments
Labels
Awaiting User Reply User needs to reply before issue can continue.

Comments

@Ferocimo
Copy link

Ferocimo commented May 1, 2020

The output of command /version on my server is: Paper version git-Paper-229 (MC: 1.15.2)
The output of command /version citizens on my server is: Citizens version 2.0.26-SNAPSHOT (build 1921)
The output of command /version sentinel on my server is: version 2.0.1-SNAPSHOT (build 343)
A pastebin.com link to my (full!) Sentinel/config.yml file is: (PLEASE FILL IN)

Hello,

We noticed several issues with latest versions of Citizens, Sentinel and Minecraft.

The first one is that Sentinels can't move in water at all, again. This issue was previously fixed but it seems to be back. We have set "avoidwater" to true but players often just push sentinels in water and since they can't get out, it's easy kill.

The second one is that Sentinels seem to mimic player movements, which means they "pause" whenever the player pauses and then resume chasing. In general, Sentinels very often pause mid chase for a moment, which makes their behavior very unrealistic and more easily predictable.

The third one is that they have troubles with carpets. If a room has carpets, the sentinel seems to think that these are "full blocks" the player walks in and they seem to think they can't pass. So they don't move at all. See below :
http://www.noelshack.com/2020-18-5-1588365487-2020-05-01-22-36-25.png

Thank you for your time.

@mcmonkey4eva
Copy link
Member

mcmonkey4eva commented May 2, 2020

water and carpets are Citizens issues, not Sentinel
-> water is CitizensDev/Citizens2#1298
-> carpet is CitizensDev/Citizens2#1171 and is theoretically resolved by just enabling use-new-finder

The pausing might be a Sentinel issue. That can happen if the Sentinel gets into attack range of the player and then the player moves away... once it's in range it's not going to continue chasing because it's already close enough. If there's more to it than that, please explain further and/or post a video to demonstrate what's happening.

@mcmonkey4eva mcmonkey4eva added the Awaiting User Reply User needs to reply before issue can continue. label May 2, 2020
@Ferocimo
Copy link
Author

Ferocimo commented May 2, 2020

Unfortunately we can't use new-finder because it causes massive performance issues. What should we do ?

For the pausing : https://streamable.com/cyxmc4
I'm not sure it's a range issue, as you can see at the beggining of the video it pauses right away with quite a distance from the player.

@mcmonkey4eva
Copy link
Member

That looks like it is a range thing - it's within attack range of the player when it stops, but the player keeps moving, and it stays stopped for a bit too long and so falls behind.
Do you have workaround entity chase pathfinder enabled in your config?

@Ferocimo
Copy link
Author

Ferocimo commented May 2, 2020

Okay I see.
No it's disabled.

@mcmonkey4eva mcmonkey4eva changed the title Various pathfinding issues in latest versions Sentinel NPCs chasing a target seem to pause when close May 2, 2020
@mcmonkey4eva
Copy link
Member

Try build 344+ from https://ci.citizensnpcs.co/job/Sentinel/ , and specifically enable workaround entity chase pathfinder.
I think it should be better about it. The basic idea, is it tracks the player's motion to predict where they're going, and thus set the target as where the player will be instead of where they are. This should make it much less likely to pause while chasing a moving player.

@Ferocimo
Copy link
Author

Ferocimo commented May 3, 2020

It seems fully functionnal now, thank you. Now we just have to wait for the Citizens improvement to new-finder to fix the rest...

@Ferocimo Ferocimo closed this as completed May 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting User Reply User needs to reply before issue can continue.
Projects
None yet
Development

No branches or pull requests

2 participants