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

[lightpreview] all geometry is pure white #368

Closed
RandomErrorMessage opened this issue Aug 4, 2023 · 5 comments
Closed

[lightpreview] all geometry is pure white #368

RandomErrorMessage opened this issue Aug 4, 2023 · 5 comments

Comments

@RandomErrorMessage
Copy link
Contributor

When using lightpreview I get the following errors upon loading a BSP, and all geometry is pure white regardless of which mode I select. I tried building from 08ae1540de5e070debf8bf94cadfc4d573ab44d2 to see if this is a recent regression but nothing changed. My iGPU has OpenGL 3.30 Core, and up to 4.6 (compatibility profile).

"GL_INVALID_OPERATION in glUniform1(uniform \"MVP\"@0 is matrix)"
"GL_INVALID_OPERATION in glUniform(location=1)"
"GL_INVALID_OPERATION in glUniform(location=2)"
"GL_INVALID_OPERATION in glUniform(location=3)"
"GL_INVALID_OPERATION in glUniform(location=4)"
"GL_INVALID_OPERATION in glUniform(location=5)"
"GL_INVALID_OPERATION in glUniform(location=6)"
"GL_INVALID_OPERATION in glUniform(location=7)"
"GL_INVALID_OPERATION in glUniform(location=8)"
[TRUNCATED]
[REPEATS UNTIL LOCATION 254 INCLUSIVE]
@ericwa
Copy link
Owner

ericwa commented Aug 5, 2023

[REPEATS UNTIL LOCATION 254 INCLUSIVE]

Suggests it might be the uniform float style_scalars[256]; causing this. Maybe too large of a uniform array?

What's the GPU, OS, GPU driver? I'm only testing on a 1070 Ti/Win11/NVIDIA 536.40 at the moment.

@Paril
Copy link
Collaborator

Paril commented Aug 5, 2023

Yeah sounds like it didn't even load the other uniforms - it's calling glUniform1(value, 0) up until 254 since that's how many styles there are. We need error checking on load I think (we don't ever validate the uniform locations).

@ericwa
Copy link
Owner

ericwa commented Aug 6, 2023

6b83f36 fixes a glsl syntax error reported on mesa; might help..

@Paril
Copy link
Collaborator

Paril commented Aug 6, 2023

Oops. Nice catch.

@RandomErrorMessage
Copy link
Contributor Author

6b83f36 fixes a glsl syntax error reported on mesa; might help..

fixed, thanks!

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

3 participants