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

Fix fluid is registered twice in crafting CPU #237

Merged
merged 6 commits into from
Nov 22, 2024

Conversation

hiroscho
Copy link

@hiroscho hiroscho commented Nov 8, 2024

Closes GTNewHorizons/GT-New-Horizons-Modpack#17853

Description, see

Yea the problem is from that PR, there is both the CraftingGridCacheFluidInventoryProxyCell and the fluid discretizer's fluid crafting inventory do the same thing of registering inserted fluids as droplets in CPUs. The Proxy is more flexible (always works, no need for discretizer, is not a fake inventory), so the fluid crafting inventory should be removed IMO
GTNewHorizons/GT-New-Horizons-Modpack#17853 (comment)

@Demiu
Copy link

Demiu commented Nov 9, 2024

I did basically the same but didn't have the time to test it back then, did now and it fixes the issue.

Another one popped up where final output of a crafting in a CPU can go into the negatives, but that is actually consistent with the behavior with items so an AE2 bug not fluid crafting bug.

@boubou19 boubou19 merged commit 3314e31 into GTNewHorizons:master Nov 22, 2024
1 check passed
lordIcocain pushed a commit to lordIcocain/AE2FluidCraft-Rework that referenced this pull request Nov 27, 2024
Co-authored-by: Martin Robertz <dream-master@gmx.net>
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.

AE2. Crafted fluid registering twice in crafting queue.
5 participants