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 fixes #10163

Merged
merged 1 commit into from
Aug 25, 2024
Merged

Pathfinding fixes #10163

merged 1 commit into from
Aug 25, 2024

Conversation

someaddons
Copy link
Contributor

Closes #10151
Closes #10147
Closes #

Changes proposed in this pull request:

  • Fix npe with bad crafting recipes after e.g. mod removal
  • Fix pathing to consider pathblocks again(no those are not diving)
  • Adjust cost calculation to add randomness first, so those cannot overshadow cost reductions
  • Improve builder work position to not go too close to the placed block, avoiding to place a block inside itself Improve Study and - Research pathing to bookcases, they now have a bigger valid range, still try to move close though

[x ] Yes I tested this before submitting it.
[ ] I also did a multiplayer test.

Review please

Fix pathing to consider pathblocks again
Improve builder work position to not go too close to the placed block, avoiding to place a block inside itself
Improve Study and Research pathing to bookcases, they now have a bigger valid range, still try to move close though
@Raycoms Raycoms merged commit 4f972e8 into version/main Aug 25, 2024
5 checks passed
@Raycoms Raycoms deleted the pathfindingfixes branch August 25, 2024 12:15
Raycoms pushed a commit that referenced this pull request Aug 25, 2024
Fix npe with bad crafting recipes after e.g. mod removal
Fix pathing to consider pathblocks again(no those are not diving)
Adjust cost calculation to add randomness first, so those cannot overshadow cost reductions
Improve builder work position to not go too close to the placed block, avoiding to place a block inside itself Improve Study and - Research pathing to bookcases, they now have a bigger valid range, still try to move close though
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants