lithium-fabric icon indicating copy to clipboard operation
lithium-fabric copied to clipboard

Caching recipes and smelting

Open Madis0 opened this issue 2 years ago • 5 comments

Is your feature request related to a problem? Please describe. I have seen some mods for Fabric that attempt to cache the crafting recipes and smelting process. I would like to know if that kind of feature is being considered in Lithium as well and if not, why.

Describe the solution you'd like A feature that would cache crafting recipes and smelting process

Describe alternatives you've considered Mods Recipe Cache, Fast Furnace, FastBench

Additional context I don't really know if this would be a good idea overall, hence I suggested it here so people relevant to game optimization can comment on it.

Madis0 avatar Oct 01 '21 04:10 Madis0

These mods have a nice optimization, which I believe also fits into lithium. However we do not want to eat other mods unless the authors themselves suggest it, because the authors might not like that, even though they chose a permissive license.

2No2Name avatar Oct 01 '21 21:10 2No2Name

However we do not want to eat other mods unless the authors themselves suggest it, because the authors might not like that, even though they chose a permissive license.

I did not quite mean bundling the mods, more of just rebuilding the feature. And my concerns were reliability (server-client sync) and sustainability (breaking with releases) of doing those changes in vanilla overall, could you or another contributor do a technical overview on that? Asking it here as you do seem to be familiar with the internals of the game.

Madis0 avatar Oct 01 '21 22:10 Madis0

I am for adding this function Since multiple installed mods work better than 100 mods

ghost avatar Oct 05 '21 06:10 ghost

Even though this mod is a forge one, this could be a valuable resource while implementing this: FastSuite

BobVarioa avatar Nov 06 '21 21:11 BobVarioa

Note that vanilla has added a last-used-recipe cache to the furnace and the campfire, so this might not be needed anymore in those cases, unless there's optimizations that can be done on top of their cache. See e.g. AbstractFurnaceBlockEntity#quickCheck (mojmap)

williewillus avatar Nov 21 '22 06:11 williewillus