Luke100000 / minecraft-comes-alive

Replaces Minecraft's villagers with humans, who can be interacted with.
GNU General Public License v3.0
70 stars 49 forks source link

Performance issue with trySpawnReaper #824

Closed zeid0ne closed 3 months ago

zeid0ne commented 6 months ago

Describe the bug I understand better villager AI can significate less performance, but i thinks it's strange that the trySpawnReaper function is what that took the more memory between all my mods

To Reproduce

  1. Install MCA and Spark
  2. Go to a village
  3. /spark profiler start
  4. Wait 2-3 min
  5. /spark profiler open

Screenshots image

Version MCA Version : 7.5.11 Minecraft version : 1.20.1 Modloader : forge 47.1.3

Latest.log latest.log

Luke100000 commented 6 months ago

Agree, 5% is too much. The reaper spawner should only trigger when fire gets placed.

zeid0ne commented 3 months ago

closing this cause no activity

Luke100000 commented 3 months ago

Most likely related to #842