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

Suppress OpenGL 128(0-2) #768

Open
AdamRaichu opened this issue Nov 19, 2023 · 6 comments
Open

Suppress OpenGL 128(0-2) #768

AdamRaichu opened this issue Nov 19, 2023 · 6 comments
Labels
mod New Fabric mod to include

Comments

@AdamRaichu
Copy link

AdamRaichu commented Nov 19, 2023

CurseForge link

Not considering UNTIL you can use Overwolf account to login; I don't want twitch.

CurseForge Mod Distribution

Unknown

Modrinth link

https://modrinth.com/mod/suppressopengl1280

Source/other link

https://github.com/AdamRaichu/suppress-opengl-1280

Mod file size

1.01 MiB

License

(any other license)

What it does

This mod prevent OpenGL debug messages with the ids 1280, 1281, or 1282 from being logged more than once in an instance.

Why should it be in the modpack

I installed this modpack for the first time today. I added Rethinking Voxels, then booted it up. Over the course of seven and a half minutes of active time, I got 57376 lines in the log. The vast majority of these were OpenGL debug errors (a couple examples are below).

[20:25:26] [Render thread/INFO]: OpenGL debug message: id=1282, source=API, type=ERROR, severity=HIGH, message='Error has been generated. GL error GL_INVALID_OPERATION in (null): (ID: 1710494744) texture 49, "" is already bound to another target'

[20:29:30] [Render thread/INFO]: OpenGL debug message: id=1280, source=API, type=ERROR, severity=HIGH, message='Error has been generated. GL error GL_INVALID_ENUM in (null): (ID: 897695363) non-integer <format> 28 has been provided.'

Let me tell you the line numbers these came from (almost exactly 4 seconds apart) in order to give you an idea of how often these were being logged. The last instance of the first line (there were 57 with the same timestamp) was on line 3955. The first instance of the second one (4 seconds later) was on line 56170!

Luckily, I had encountered a similar problem before, and decided to solve it with this simple mod.

Why shouldn't it be in the modpack

Is not on CurseForge. Also, the title may be confusing to some players, because, while the mod does not actually fix any errors, its intended purpose is to prevent errors from being logged.

Additional details

Full disclosure: I am the author of this mod. It being included in fabulously optimized would probably be great for me. However, I do think that it would be helpful for those trying to actually read the log. Looking through tens of thousands of lines of code to find what you are looking for is impractical. Also, users who want to see those errors (shader authors for example) can use the built in config to turn off suppression.

@AdamRaichu AdamRaichu added the mod New Fabric mod to include label Nov 19, 2023
@Madis0
Copy link
Member

Madis0 commented Nov 19, 2023

Hmm. I think FO already has the feature through Sodium/Iris/Sodium Extra. Check the video settings.

If not, it would make sense to implement it in one of the three.

@Madis0
Copy link
Member

Madis0 commented Nov 20, 2023

@AdamRaichu
Copy link
Author

pilt CaffeineMC/sodium-fabric@dev/src/main/java/me/jellysquid/mods/sodium/mixin/core/WindowMixin.java#L16

What do you think of that option? Is that different?

That is showing as disabled for me.

image

I am still getting the error in the log. Adding my mod prevents it from being spammed.

@Madis0
Copy link
Member

Madis0 commented Nov 21, 2023

Perhaps it's worth finding out why the option is disabled for you then.

@AdamRaichu
Copy link
Author

pilt CaffeineMC/sodium-fabric@dev/src/main/java/me/jellysquid/mods/sodium/mixin/core/WindowMixin.java#L16

What do you think of that option? Is that different?

Confirmed on a different device: With Performance > Use No Error Context checked, still getting the error spam. (Confirmed both with Rethinking Voxels and Complementary Reimagined.)

Side note: Log monitor in Modrinth launcher seems to be ignoring duplicate message/timestamp combinations; good on them.

@Madis0 Madis0 changed the title Mod Request: Suppress OpenGL 128(0-2) Suppress OpenGL 128(0-2) Jan 10, 2024
@AdamRaichu
Copy link
Author

Update: Mod is now available on CurseForge (pending mod approval).

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

No branches or pull requests

2 participants