TeamMFR / MineFantasyReforged

GNU General Public License v3.0
18 stars 20 forks source link

[Bug] Heavy RAM load during JEI indexing #50

Open ALJ323 opened 1 year ago

ALJ323 commented 1 year ago

Is there an existing issue for this?

Observed behaviour

When trying out this mod and adding it to a pack containing JEI, the memory being used by this mod grows exponentially, even moreso when using other mods that this mod has compatible materials for i.e. Metallurgy. Its almost double, in my case with a modpack of ~60 mods and a 4GB limit its going from 2GB on a normal load to 3.5GB when in usage. A pretty critical spike even at this scale.

These problems don't persist with one or the other being disabled (Minefantasy or JEI), and the mod loads normally up until step 6 of 7 when JEI is stated to be indexing ingredients. However, being able to play without JEI is an issue, both regarding being able to figure out how to do anything, as well as customizing the experience.

Out of curiosity I tried the mod under heavier conditions, adding it to a larger modpack with about 200 mods that loads to about 4GBs and it ended up ballooning up to 7 Gigs out of 8, it takes up even more when increasing the memory limit further (more than minecraft usually does when doing so).

Expected behaviour

Stable memory usage when trying to use this mod alongside others

Steps to reproduce

Place Minefantasy in a modpack environment regardless of size launch the game

Crashlog

No response

Environment (Singleplayer/Server, etc.)

Issue present on both sides

Mod version

0.2.0

Forge version

1.12.2 - 14.23.5.2860

Other mods

!mixinbooter-4.2 AbyssalCraft-1.12.2-1.10.3 AutoRegLib-1.3-32 Baubles-1.12-1.5.2 BloodMagic-1.12.2-2.4.3-105 BNBGamingCore-1.12.2-0.12.0 BNBGamingLib-1.12.2-2.17.6 Botania+r1.10-364.4 CarianStyle-1.12.2-1.0.8 CodeChickenLib-1.12.2-3.2.3.358-universal Commons0815-1.12.2-1.4.0 CraftTweaker2-1.12-4.1.20.662 DimensionalControl-1.12.2-2.13.0 elementalsorcery-0.14.0 EmbersRekindled-1.19 enchantedwolves-1.12.2-1.1 EnchantmentDescriptions-1.12.2-1.1.20 forestry_1.12.2-5.8.2.422 Forgelin-1.8.4 fossilsarcheology-8.0.5 geckolib-forge-1.12.2-3.0.30 GottschCore-mc1.12.2-f14.23.5.2859-v1.15.0 Guide-API-1.12-2.1.8-63 HeatAndClimateLib_1.12.2-3.8.0 HeatAndClimateMod_1.12.2-3.8.0 iceandfire-1.9.1-1.12.2 jei_1.12.2-4.16.1.302 jeiintegration_1.12.2-1.6.0 JustEnoughIDs-1.0.3-55 libnine-1.12.2-1.2.1 llibrary-1.7.20-1.12.2 loliasm-5.3 Mantle-1.12-1.3.3.55 Metallurgy-4-Reforged-1.3.0 MineFantasyReforged-1.12.2-0.2.0 MTLib-3.0.7 mup-1.3.9 NuclearCraft-2o.5.5-1.12.2 parry-1.0-hotfix Patchouli-1.0-23.6 Psi-r1.1-78.2 railcraft-12.1.0-beta-2 rats-3.2.21-1.12.2 techguns-1.12.2-2.0.2.0_pre3.2 Thaumcraft-1.12.2-6.1.BETA26 ThaumicAugmentation-1.12.2-2.0.9 theaurorian-1.12.2-release-may3021 TheBetweenlands-3.7.3-universal theoneprobe-1.12-1.4.28 Treasure2-mc1.12.2-f14.23.5.2859-v2.1.0 twilightforest-1.12.2-3.11.1021-universal WitcheryResurrected-1.12.2-0.5.2.4-forge-all.jar (disabled)