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

Default config is incompatible with Fabrication minor mechanic #517

Open
skymen opened this issue May 22, 2024 · 1 comment
Open

Default config is incompatible with Fabrication minor mechanic #517

skymen opened this issue May 22, 2024 · 1 comment
Labels
T-bug Type: Bug

Comments

@skymen
Copy link

skymen commented May 22, 2024

Hi,

Fabrication got a new feature recently for refueling furnace minecarts with hoppers, however it seems to be incompatible with lithium's default config for hopper optimisations.

FalsehoodMC/Fabrication#718

In the config, disabling this rule fixes the issue: mixin.block.hopper
However, it seems that this disables every hopper optimisations when realistically, only this specific optimisation seems to be at fault:
https://github.com/CaffeineMC/lithium-fabric/blob/44f4330b9ab1cef568e80336d949c61386f747b9/src/main/java/me/jellysquid/mods/lithium/mixin/block/hopper/AbstractMinecartEntityMixin.java

(this is a conjecture from reading the code, I didn't actually debug the game or remove the optimisation manually to see if it fixes it)

Here is the relevant code from the Fabrication mod that's affected by the issue:
https://github.com/FalsehoodMC/Fabrication/blob/3.0/1.18/src/main/java/com/unascribed/fabrication/mixin/d_minor_mechanics/furnace_minecart_resupplying/MixinHopperBlockEntity.java#L31


Version Information

lithium-fabric-mc1.20.1-0.11.2

Expected Behavior

Fabrication method should be triggered without having to disable every other optimisation, and ideally there should be a way for Fabrication to ask Lithium to change a config rule

Actual Behavior

User has to disable all hopper optimisations for it to work

Reproduction Steps

1- Install both mods
2- Enable "refueling furnace minecart" minor mechanic in Fabrication config
3- Place a rail down with a hopper with fuel pointing into it
4- Place furnace minecart on rail

Other Information

Any info not outlined here can be found in the original Fabrication issue
FalsehoodMC/Fabrication#718

@skymen skymen added the T-bug Type: Bug label May 22, 2024
@2No2Name
Copy link
Member

Thank you for your report.

Lithium has another feature that makes block entities (including hoppers) sleep under certain conditions, with certain events waking them up again (e.g. an inventory entity moving nearby). This might be incompatible with the additional hopper feature, because the sleeping and waking conditions are not implemented. I suggest testing with the line mixin.world.block_entity_ticking.sleeping.hopper=false added to the lithium.properties file in the config folder in the .minecraft folder. If that fixes the issue, please mention it here or in the original issue report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-bug Type: Bug
Projects
None yet
Development

No branches or pull requests

2 participants