Futurepack-API icon indicating copy to clipboard operation
Futurepack-API copied to clipboard

EScanner Rendering crash [1.16.5]

Open DamondSword2247 opened this issue 1 year ago • 11 comments

Describe the bug Rendering crash when rendering the crafting recipes in the EScanner.

To Reproduce Steps to reproduce the behavior:

  1. Open EScanner.
  2. Navigate to TechTree.
  3. Open any of the item pages and scroll down to the crafting recipe.
  4. Game Crashes.

Screenshots Can't provide as it locks up and crashes.

Additional context Tested in a fresh mod pack for the express purpose of testing only futurepack alone, to try and recreate the EScanner rendering crash I was having in another mod pack. Default graphics settings.

Version Minecraft Version: 1.16.5 Forge Version: 36.2.34 Futurepack Version: Futurepack-1.16.5-31.3.7008

Logs https://pastebin.com/E6Ft7p90

DamondSword2247 avatar Jun 24 '24 02:06 DamondSword2247

Having the same issue, same version. After further analysis of the crash report, it looks like (don't take my word for this) the E-scanner is having issues rendering items in the object description. Is there a way to disable this as a workaround?

Update: Currently scavenging through the code to figure out how to shut down recipe rendering.

Warentan avatar Jun 30 '24 23:06 Warentan

I hope this can be fixed because the escanner is almost useless because of this

DamondSword2247 avatar Jul 02 '24 15:07 DamondSword2247

I hope this can be fixed because the escanner is almost useless because of this

Did you use modpack while playing futurepack

Ashwinaaaaaaahdt avatar Aug 03 '24 01:08 Ashwinaaaaaaahdt

Repeatable on: Mod Version 1.18.2-33.0.7544 Forge Version 40.2.10 MC Version 1.18.2

My Crash Report: crash-2024-08-08_07.32.07-client.txt

My perspective: Somehow, the index is going into the negatives (which is not permitted in Java) within one of the methods in this crash report. What I would do is double check the 5 FP methods listed.

thebloxxer11 avatar Aug 08 '24 14:08 thebloxxer11

Repeatable on: Futurepack-1.12.2-26.4.89 Forge Version: 14.23.5.2860 MC Version 1.12.2

My Crash Report: https://mclo.gs/Y9CkRHY

Really anoying cuz i cant play the mod now unless i constantly use the wiki

Gurasic avatar Aug 18 '24 12:08 Gurasic

Further testing resulted in Any research with a recipe that uses a tag instead of an item ID will crash. Tested with an up-to-date Forge in 1.18.2

thebloxxer11 avatar Aug 19 '24 18:08 thebloxxer11

In that case, my assumptions (or rather main hypothesis) are: something is trying to access an array using the recipe identifier as index, and when using a (I presume string?) tag, the execution explodes.

But I do not know it for sure at all, since I do not know how to debug Minecraft. Do you know how could I debug the mod? I normally use vs code as tool for programming.

Somersault0023 avatar Aug 19 '24 19:08 Somersault0023

I have submitted a fix for this issue. There is a function in the recipe rendering code that takes the current Unix timestamp in milliseconds and divides by 800. Starting on June 10, 2024, the output value exceeded the Java integer limit. This led to crashes when invalid array indices were produced.

Hypermoron avatar Sep 08 '24 23:09 Hypermoron

I have submitted a fix for this issue. There is a function in the recipe rendering code that takes the current Unix timestamp in milliseconds and divides by 800. Starting on June 10, 2024, the output value exceeded the Java integer limit. This led to crashes when invalid array indices were produced.

@Hypermoron @DamondSword2247 @thebloxxer11 Sadly the developer can't implement the feature right this year. This is due to lack of time and developers for this. He is alone. If you badly wanted an update for futurepack every time, you can help him to develop the mod. I am looking for five coders to help him. If 5 coders come to help him, then he would do updates on the mods. Besides he is working alone

Ashwinaaaaaaahdt avatar Sep 09 '24 09:09 Ashwinaaaaaaahdt

Fixed in 1.18.2 https://legacy.curseforge.com/minecraft/mc-mods/futurepack/files/5712819 https://modrinth.com/mod/futurepack/version/33.0.7547

mcenderdragon avatar Sep 11 '24 07:09 mcenderdragon

Umm, he made me wrong

Ashwinaaaaaaahdt avatar Sep 30 '24 01:09 Ashwinaaaaaaahdt