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

Enabling 2x2 stairs causes server crash 1.12.2 #2

Closed
PurpleHel opened this issue Nov 7, 2017 · 6 comments
Closed

Enabling 2x2 stairs causes server crash 1.12.2 #2

PurpleHel opened this issue Nov 7, 2017 · 6 comments

Comments

@PurpleHel
Copy link

Setting stairs to false in config fixes it. Wasn't really using the 2x2 recipe anyway, so don't care enough to figure out if it's just PiTweaks or if it's an interaction with another mod, but sharing the crash report is pretty painless, so here you go.

https://pastebin.com/s9yggxus

Unrelated, searching PiTweaks on curseforge doesn't turn up your mod, which is minorly inconvenient. :)

@CplPibald
Copy link
Owner

Looks like there's a mod adding a recipe that's returning an empty list for getMatchingStacks(), and that's making the stair scanning code blow up.

Thanks for bringing this to my attention.

@PurpleHel
Copy link
Author

My full mods list in a slightly more readable form than the crash report list can be found at https://pastebin.com/yaz3V5rT

I'm willing to do some minor testing of the 'hey, disable such-and-such mod and see if it still crashes?' sort. Just not up to trying to hunt down for myself if it was mod interaction or internal. :)

@CplPibald
Copy link
Owner

I don't think it matters which mod it's interacting with. As long as Forge allows it, some mod will mess it up. PiTweaks needs to be smarter about ignoring malformed recipes when scanning.

Can you check if you still see the same crash with this version? If that fixes it, I'll upload it to curseforge. Thanks.

https://www.dropbox.com/s/sa0r5rqnsjlf636/PiTweaks-1.12-0.5.1.jar?dl=1

@PurpleHel
Copy link
Author

PurpleHel commented Nov 8, 2017

Checking! :)

Edit-Yup, works just fine! No crashes, I even made some 2x2 stairs with no problems.

I did notice that the 2x2 stairs recipe doesn't show up when you hit recipes for stairs in JEI/NEI.

https://imgur.com/gallery/BbeHL

@CplPibald
Copy link
Owner

You said this is on a server? Did you also update your client config to turn the tweak back on?

I noticed in my own testing the recipe works as long as the 2x2 tweak is enabled on the server, but for it to show up in JEI, the tweak has to also be enabled on the client.

@PurpleHel
Copy link
Author

Ah, ok, cool. I did not update the client config, so that'd be the problem with it not showing up in JEI. :)

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

No branches or pull requests

2 participants