Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for grass in fluids #84

Merged
merged 7 commits into from
Nov 17, 2023

Conversation

paulevsGitch
Copy link
Contributor

Should fix bonemeal grass in fluids. Due to impossibility to launch StAPI after all updates I can't test it. Should work

@mineLdiver
Copy link
Member

Flowers still seem to break fluids
2023-11-14_20 53 12

@paulevsGitch
Copy link
Contributor Author

Fixed

@mineLdiver
Copy link
Member

-32 to 128 world still crashes:

java.lang.ArrayIndexOutOfBoundsException: Index 40960 out of bounds for length 40960
	at net.minecraft.class_538.method_1798(class_538.java:112)
	at net.minecraft.class_538.method_1806(class_538.java:207)
	at net.minecraft.class_326.method_1807(class_326.java:53)
	at net.minecraft.class_326.method_1806(class_326.java:78)
	at net.minecraft.world.World.method_214(World.java:311)
	at net.minecraft.world.World.getBlockId(World.java:265)
	at net.minecraft.world.World.handler$zek000$station-flattening-v0$fixHeightSearch(World.java:3076)
	at net.minecraft.world.World.method_152(World.java)
	at net.minecraft.world.dimension.Dimension.method_1770(Dimension.java:57)
	at net.minecraft.world.World.method_212(World.java:170)
	at net.minecraft.world.World.<init>(World.java:153)
	at net.minecraft.world.World.<init>(World.java:134)
	at net.minecraft.client.Minecraft.method_2120(Minecraft.java:1098)
	at net.minecraft.client.gui.screen.world.CreateWorldScreen.buttonClicked(CreateWorldScreen.java:97)
	at net.minecraft.client.gui.screen.Screen.mouseClicked(Screen.java:69)
	at net.minecraft.client.gui.screen.world.CreateWorldScreen.mouseClicked(CreateWorldScreen.java:116)
	at net.minecraft.client.gui.screen.Screen.onMouseEvent(Screen.java:110)
	at net.minecraft.client.gui.screen.Screen.tickInput(Screen.java:99)
	at net.minecraft.client.Minecraft.tick(Minecraft.java:937)
	at net.minecraft.client.Minecraft.run(Minecraft.java:531)
	at java.base/java.lang.Thread.run(Thread.java:840)

@paulevsGitch
Copy link
Contributor Author

Attempt to fix that

@mineLdiver
Copy link
Member

Same crash

java.lang.ArrayIndexOutOfBoundsException: Index 40960 out of bounds for length 40960
	at net.minecraft.class_538.method_1798(class_538.java:112)
	at net.minecraft.class_538.method_1806(class_538.java:207)
	at net.minecraft.class_326.method_1807(class_326.java:53)
	at net.minecraft.class_326.method_1806(class_326.java:78)
	at net.minecraft.world.World.method_214(World.java:311)
	at net.minecraft.world.World.getBlockId(World.java:265)
	at net.minecraft.world.World.handler$zen000$station-flattening-v0$fixHeightSearch(World.java:4076)
	at net.minecraft.world.World.method_152(World.java)
	at net.minecraft.world.dimension.Dimension.method_1770(Dimension.java:57)
	at net.minecraft.world.World.method_212(World.java:170)
	at net.minecraft.world.World.<init>(World.java:153)
	at net.minecraft.world.World.<init>(World.java:134)
	at net.minecraft.client.Minecraft.method_2120(Minecraft.java:1098)
	at net.minecraft.client.gui.screen.world.CreateWorldScreen.buttonClicked(CreateWorldScreen.java:97)
	at net.minecraft.client.gui.screen.Screen.mouseClicked(Screen.java:69)
	at net.minecraft.client.gui.screen.world.CreateWorldScreen.mouseClicked(CreateWorldScreen.java:116)
	at net.minecraft.client.gui.screen.Screen.onMouseEvent(Screen.java:110)
	at net.minecraft.client.gui.screen.Screen.tickInput(Screen.java:99)
	at net.minecraft.client.Minecraft.tick(Minecraft.java:937)
	at net.minecraft.client.Minecraft.run(Minecraft.java:531)
	at java.base/java.lang.Thread.run(Thread.java:840)

@paulevsGitch
Copy link
Contributor Author

Now everything should work fine

@mineLdiver mineLdiver merged commit 17cf9a7 into ModificationStation:master Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants