TerraFirmaGreg-Team / Modpack-Modern

An innovative modpack that contains GregTech and TerraFirmaCraft on 1.20.x.
GNU General Public License v3.0
37 stars 9 forks source link

Crash on making plates with mechanical press #202

Closed Wyrmlet closed 5 months ago

Wyrmlet commented 6 months ago

Version modpack Used

0.6.3

Environment

Singleplayer

Side Crashed

Server

New Worlds

Unsure

Steps to Reproduce

Using the mechanical press recipes for plates sometimes crashes the game. After the crash, the plate is not created, but the ingot is consumed.

Crash Report

crash-2024-01-19_18.45.01-server.txt latest.log

Additional Information

This feels like a bug with the chance of no output in the recipe, as the plate is not created after the world is reloaded.

Exzept1on commented 6 months ago

Needs testing after 0.6.4

Wyrmlet commented 5 months ago

Tested on 0.6.6, an ingot was voided on the press but the game did not crash. Probably fixed.

C0ffo commented 5 months ago

У меня тоже что-то с пропажей под прессом,кинул под пресс листы делатся и когда через 5 мин пришел то их небыло нигде

Wyrmlet commented 5 months ago

Still happening as of 0.6.8, but not always. Sometimes plates are correctly voided, but sometimes the game crashes instead. Sometimes the plates are created after reloading, so maybe this is not connected to the chance output at all? crash-2024-01-25_14.01.02-server.txt latest.log

Exzept1on commented 5 months ago

ok, thanks for report

Exzept1on commented 5 months ago

Needs testing after 0.6.9, idk why this shit appers :(

Wyrmlet commented 5 months ago

From both server crash logs: java.lang.NullPointerException: Cannot assign field "locked" because "this.heldItem" is null Seems like it has something to do with the create depot logic and fails specifically if I don't have any item held? In any case, I ended up moving on to using a bender instead, so I'm not sure if I'll end up testing this as I play casually.

Wyrmlet commented 5 months ago

https://github.com/Creators-of-Create/Create/blob/2201d87da7744cf9ee3f83e1e4c21643244f342b/src/main/java/com/simibubi/create/content/logistics/depot/DepotBehaviour.java#L147

Looked through the relevant code, seems to be a create issue specifically? The tick update seems to lack a check for if the item held is null, and it tries to assign a value to it at line 147 regardless, which causes the null pointer exception. I'll try bringing this up in an issue on their github, maybe it'll get fixed on their end.

Wyrmlet commented 5 months ago

Started a new issue on https://github.com/Creators-of-Create/Create/issues/6053

Wyrmlet commented 5 months ago

According to the people in the create repo, this might be happening because mods for chunk claiming can interfere with how create works in some places. This needs to be tested with claimed chunks and without, as well as with and without the relevant mods. I did have my base claimed and chunkloaded, so that might have caused the issue.

Wyrmlet commented 5 months ago

For context, each time this happened I remember looking at the depot, it never happened while I was away. It also only seems to happen when my hands are empty. Hopefully this helps to verify this.

Exzept1on commented 5 months ago

In any case, I assume that the crash is not related specifically to TerraFirmaGreg or its recipes, perhaps this is really an OpenClaims mod, in which case I will close it, we as modpack developers cannot fix it