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.

Sign up for GitHub

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

Jump to bottom

Several navigation fix #10411

Open
wants to merge 11 commits into
base: version/1.21
Choose a base branch
from
Open

Several navigation fix #10411

wants to merge 11 commits into from
+57 −10

Conversation

HexatomicRing
Copy link

  • Path-finding can now identify the direction of stairs, avoiding jumping up stairs from the side.
  • Path-finding can now identify doors. Citizens and intruders now prefer paths with fewer doors to open.
  • Modified the drop cost in path-finding. 1 falling damage cost 25 in addition. Swimming enter cost is lowered so that citizens will drop in water to avoid falling damage. (In practical situations, it may fail because citizens sometimes move too fast to drop into the water)

Testing

  • [√] Yes I tested this before submitting it.
  • [×] I also did a multiplayer test.

Review please
// Sorry that I deleted my fork by mistake, so I had to re-create this pull request😭.

@HexatomicRing
Some improvements in navigation
bd37b31 
Validated the facing of stairs while computing jumping cost.
Modified some path-finding cost to be closer to its real time cost.
Added path-finding cost for opening a door.
@HexatomicRing
Update AbstractPathJob.java
85eb19a
@HexatomicRing
Update AbstractPathJob.java
5b3c94f
@HexatomicRing
Update AbstractPathJob.java
9b923ae 
Simplified the stair validation logic
@HexatomicRing
Merge branch 'version/1.21' into HexatomicRing-Navigation-Fix
b764da9
@HexatomicRing
More door cases
2e1aed7
@HexatomicRing
Dropping modification
46e06eb
@HexatomicRing
Merge branch 'version/1.21' into HexatomicRing-Navigation-Fix
f738cb2
@HexatomicRing
Corrected format
acb9857
@HexatomicRing
Merge branch 'HexatomicRing-Navigation-Fix' of https://github.com/Hex…
ab82b60 
…atomicRing/minecolonies into HexatomicRing-Navigation-Fix
@HexatomicRing
Merge branch 'version/1.21' into HexatomicRing-Navigation-Fix
2987e1f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@HexatomicRing