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

JEI rendering errors [bug] [high importance] #260

Closed
DanHolli opened this issue Sep 20, 2016 · 14 comments
Closed

JEI rendering errors [bug] [high importance] #260

DanHolli opened this issue Sep 20, 2016 · 14 comments
Labels

Comments

@DanHolli
Copy link

whole original discussion with the creator of JEI about the problem

JustEnoughItems issue #372

@DanHolli
Copy link
Author

until this is solved i won't be using optifine (witch I hate to need to do)

@DanHolli DanHolli changed the title JEI rendering errors JEI rendering errors [bug] [high importance] Sep 20, 2016
@sp614x
Copy link
Owner

sp614x commented Sep 20, 2016

Is this reproducible with only OptiFine and JEI installed (no other mods)?

@mezz
Copy link

mezz commented Sep 20, 2016

Here's a crash log with fewer mods, but I haven't seen anyone try to reproduce it with a minimal setup yet.
https://gist.github.com/mightydanp/06dbd8f674ac06ccb5e9f7025c59137b

JEI copies some of the Minecraft item rendering code into its own classes with some small changes in order to render the item list with fewer OpenGL context changes. If Optifine is changing that code in Minecraft and expects it to work a certain way, it may be relevant to this bug.

Rendering code starts here:
https://github.com/mezz/JustEnoughItems/blob/1.10/src/main/java/mezz/jei/gui/ingredients/GuiItemStackFastList.java#L96-L161
and the copied stuff starts here:
https://github.com/mezz/JustEnoughItems/blob/1.10/src/main/java/mezz/jei/gui/ingredients/GuiItemStackFast.java#L74-L105

I also have a fairly small AccessTransformer here:
https://github.com/mezz/JustEnoughItems/blob/1.10/src/main/resources/jei_at.cfg

@sp614x
Copy link
Owner

sp614x commented Sep 21, 2016

The "IllegalStateException: Already building!" is a follow-up error.
Something else have crashed already and because of this the VertexBuffer is still open and the next code which tries to use the VertexBuffer gets the "Already building" error.
The previous error may be visible in the error log.

@DanHolli
Copy link
Author

DanHolli commented Sep 24, 2016

it only happens for me with optifine installed
(I dislike the VERY convinent 'Close and comment' button)

@DanHolli DanHolli reopened this Sep 24, 2016
@sp614x
Copy link
Owner

sp614x commented Sep 25, 2016

@DanHolli123 Error log with only OptiFine and JEI?
Steps to reproduce?

@ghost
Copy link

ghost commented Oct 10, 2016

When using only OptiFine_1.10.2_HD_U_C3 and jei_1.10.2-3.12.6.304 on Forge 2099 this error does not occur for me. However I've had it happen when searching for items, earlier today in fact, on a larger mod order. So I'll probably spend a bit of time going through the mods in my load order manually and finding which recipes this happens for. No guarantees on timeliness however.

@sp614x sp614x added the mods label Oct 10, 2016
@ghost
Copy link

ghost commented Oct 10, 2016

The only mod that has thrown an error for me, when used in conjuction with the versions of Optifine and JEI listed above, so far is Inductive Automation (and its dependency library, which has a long and complicated name that I don't have with me atm). I'll try reproducing the crash on a minimal load order later and report the crash to IA's mod author meanwhile.

@ghost
Copy link

ghost commented Oct 12, 2016

okay, so IA is definitely one mod that causes issues, but I think that by hiding its items from the JEI menu this crash can be bypassed. obviously this isn't a proper workaround, and there may be more mods that have errors with JEI and Optifine.

to clarify, JEI and Optifine work together fine; and, JEI and IA work together fine (I haven't tested only IA and Optifine). it seems to be that when the three are combined that a crash occurs. whose end is this on?

@sp614x
Copy link
Owner

sp614x commented Oct 12, 2016

In order to be fixed the bug has to be reproducible with the minimal number of mods involved.
Best would be only 2 mods (optifine and another), but sometimes there are 3-mod dependancies.

@ghost
Copy link

ghost commented Oct 12, 2016

With only Optifine, IA and its dependency (CD4017BE_lib-1.10.2-4.2.0) loaded, I get an identical crash when attempting to navigate to IA's creative tab in creative mode. The crash does not occur when loading only Optifine & CD4017BE_lib. The crash does not occur with Optifine disabled. JEI, then, seems to be unrelated to this case. I believe the words required here are, "my bad". xP

Crash: http://pastebin.com/raw/E2cFHdBT
Log: http://puu.sh/rGV7N/0f6a9b4542.log

CD4017BE/InductiveAutomation#17

@sp614x
Copy link
Owner

sp614x commented Oct 13, 2016

Tested MC 1.10.2 + Forge #2099 + preview OptiFine C4 + IA 5.1.1 + CD4017BE_lib 4.2.0.
Not crashing when browsing items in the IA creative tab, everything seems to work.
The error log shows missing model variants, but this also happens without OptiFine.
Can you test again with the latest preview OptiFine C4?

@ghost
Copy link

ghost commented Oct 13, 2016

Fantastic, the preview C4 works perfectly (as far as I can tell). Thanks for fixing that so quickly!

@sp614x sp614x closed this as completed Oct 13, 2016
@DanHolli
Copy link
Author

will try soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants