search

CaffeineMC / lithium-fabric

A Fabric mod designed to improve the general performance of Minecraft without breaking things
GNU Lesser General Public License v3.0
1.94k stars 190 forks source link

Incompatible with modded POI Type registrations on Neoforge #594

Open lostmyself8 opened 5 days ago

lostmyself8 commented 5 days ago

Version Information

lithium-neoforge-mc1.21.1-0.14.2

Expected Behavior

Villagers in other mods(for example, AE2, ie, and PneumaticCraft) should receive their professions. Work blocks(other mods) should be able to be placed in the trader of Simple-Villagers.

Actual Behavior

Villagers in other mods cannot receive professions. I cannot put work blocks(other mods) into the trader of Simple-Villagers.

Reproduction Steps

Load lithium and other mods, such as AE2, and then place charger and villagers,and you will find that they will not receive work block.

Other Information

Forgive me for using translation software.

2No2Name commented 3 days ago

Possibly related to https://github.com/CaffeineMC/lithium-fabric/issues/582

2No2Name commented 3 days ago

Workaround: Add the line mixin.ai.poi=false to the lithium.properties file in the config folder and restart the game.

2No2Name commented 3 days ago

This didn't have anything to do with simple-villagers, but just lithium and neoforge's poi registration technique

lostmyself8 commented 3 days ago

Thank you for your answer. It is indeed useful. But after the change, the villagers' transactions became chaotic 2024-11-28_15 53 27

2No2Name commented 3 days ago

Which block is needed to get a machinist trader in your screenshot

lostmyself8 commented 3 days ago

Engineer's Circuit Workbench from Immersive Engineering

lostmyself8 commented 3 days ago

Oh, I searched and it seems to be an issue with immersion engineering. Sorry

2No2Name commented 3 days ago

Probably referring to https://github.com/BluSunrize/ImmersiveEngineering/issues/6109