JDKDigital / productive-bees

Useful bees
https://www.curseforge.com/minecraft/mc-mods/productivebees
Other
64 stars 42 forks source link

Adding compatibility with Minecolonies #429

Closed Niduin closed 6 months ago

Niduin commented 1 year ago

The Forester from Minecolonies cannot chop the Nest - like Oak Nest - from the trees logs. Would it bee possible to add something like a option in the config, that the nest cannot spawn in specific trees ?

JaisDK commented 1 year ago

Don't have flowers near the tree farm

Niduin commented 1 year ago

well... when it is this simple - thank you :-D

Which radius for the flowers is needed around a tree that a nest cannot spawn?

JaisDK commented 1 year ago

it's a 2-3 blocks radius around the sapling that has to be clear of flowers

thinking about it, cherry leaves are considered flowers, so I can see an issue growing those

JaisDK commented 1 year ago

mangrove propagules are also flowers, which is just silly

Niduin commented 1 year ago

But Oak, birch, spruce, jungle are not?

JaisDK commented 1 year ago

No just the mangrove

modsbydreamCritting commented 6 months ago

Adding the nest blocks to a tag at data/minecolonies/tags/blocks/tree.json would allow the forester to chop them down as part of the tree.

JaisDK commented 6 months ago

I added them to minecraft:logs not too long ago which is included in the minecolonies tree tag.

Devilswarchild commented 4 months ago

I added them to minecraft:logs not too long ago which is included in the minecolonies tree tag.

the forester still ignores them

JaisDK commented 4 months ago

well then there's nothing I can do about it, adding it to the tag should pass this check but if it's still not considered part of the tree then it's on them to fix https://github.com/ldtteam/minecolonies/blob/version/main/src/main/java/com/minecolonies/core/entity/ai/workers/util/Tree.java#L146

Devilswarchild commented 4 months ago

well then there's nothing I can do about it, adding it to the tag should pass this check but if it's still not considered part of the tree then it's on them to fix https://github.com/ldtteam/minecolonies/blob/version/main/src/main/java/com/minecolonies/core/entity/ai/workers/util/Tree.java#L146

ok started an issue thread with them linking back to this thread