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

feat: amethyst golem tweaks #1545

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Vonr
Copy link
Contributor

@Vonr Vonr commented Dec 23, 2024

makes the golems not respect claims - current behaviour causes them to drop the items but not break the block which results in an unintentional buff.

also changes their harvesting behaviour:

if they do not have a home (not sure if even possible) or their home is not an inventory, they will drop the drops as item entities.

if they do have a home and it is an inventory, they will insert the dropped items directly into the home if there is free space or skip harvesting the cluster otherwise. it will continue trying other clusters in the latter case.

Vonr added 2 commits December 23, 2024 19:59
makes the golems not respect claims - current behaviour causes them to
drop the items but not break the block which results in an unintentional
buff.

also changes their harvesting behaviour:

if they do not have a home (not sure if even possible) or their home is
not an inventory, they will drop the drops as item entities.

if they do have a home and it is an inventory, they will insert the
dropped items directly into the home if there is free space or skip
harvesting the cluster otherwise. it will continue trying other clusters
in the latter case.
@Vonr
Copy link
Contributor Author

Vonr commented Dec 23, 2024

added a commit to encourage amethyst golems to stay near their homes while pathfinding after being suggested to look into it by jarva

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant