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

Block with snow #9

Open
daretmavi opened this issue Jun 16, 2020 · 1 comment
Open

Block with snow #9

daretmavi opened this issue Jun 16, 2020 · 1 comment
Labels
bug movement for issues related to pathfinding or movement

Comments

@daretmavi
Copy link

Describe the bug
Villager can not jump on Block with snow on it.

To Reproduce
0. Generate word with snow (like Seed 6422333087495876490, V7 generator)

  1. Create villager and let her to follow you
  2. Get on the block with snow - villager is just standing, not jumping
  3. Remove the snow from the block - villager is following you again

Expected behavior
Ignore snow and just jump on it.

Desktop (please complete the following information):

  • Minetest version: 5.2.0
  • Other mods activated: smart inventory
@theFox6
Copy link
Owner

theFox6 commented Jun 16, 2020

I could actually reproduce this failure.
I experienced this problem when the villager is standing on a snow cover and trying to jump up one block onto another snow cover.

The problem is due to the simple obstacle detection.
The villager checks if there is an obstacle in front of him, if there is one the villager jumps.
But only if the block above the one in front of him is not an obstacle.
The snow cover is a collidable block, this means that the villager considers the height too large to jump.

To properly detect obstacles the villager would have to consider the actual altitude and the nodeboxes and orientations of three blocks in front of him.
The one right in front, the one above and the one above that one.
This may slightly affect the performance.

I will try to add the detection. However adding this feature may take a bit of time.

@theFox6 theFox6 added the movement for issues related to pathfinding or movement label Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug movement for issues related to pathfinding or movement
Projects
None yet
Development

No branches or pull requests

2 participants