Jump to content

  • Log In with Google      Sign In   
  • Create Account


proper glUniform use


Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

  • You cannot reply to this topic
10 replies to this topic

#1 krienie   Members   -  Reputation: 102

Like
0Likes
Like

Posted 23 August 2011 - 01:30 PM

Hey all,

I'm kind of new to openGL and GLSL, so I"m not that familiar with all the proper uses yet. Like glUniform.

So here's my sitiuation: I'm currently working on a game engine that uses openGL.
I've made a DisplayObject that can render out 3D object from data stored in VBO's and a GLSL shader. It also uses a home-made matrix class for storing model-view, normal and modelview-projection matrices which the GLSL shader can use.

I"m currently reading through the OpenGL Shading Language book (orange book) and in there the writers state that uniform attributes should only be used with data that doesn't change frequently. So what I did is declare the matrices in the shader as uniforms and made a neat updateMatrices() function for my DisplayObject class that is called everytime a transformation is done to the object (for example when it translates or rotates). This function updates the matrices and then sends the new information to the GLSL shader using glUniform. Now this all works just fine when using one object, but when I introduce a second object or more, all the objects are snapped to the coordinates and orientation of the object that did the last transformation.
After some time of debugging I was able to fix this by calling chop the glUniform calls out of updateMatrices and call just before every object is drawn instead. Even though this works, I'm really doubting if I did the right thing here, because what I'm now doing is sending matrix data to the shader even though the values didn't change.

void DisplayObject::render()
{
    if ( readyToRender )
    {
        glBindVertexArray(vaoName);

        /* send new matrices to the shaderprogram */
        glUniformMatrix4fv(glGetUniformLocation(program, "mvMatrix"), 1, GL_FALSE, modelmatrix.data);
        glUniformMatrix4fv(glGetUniformLocation(program, "normMatrix"), 1, GL_FALSE, normalMatrix.data);

        /* send new modelview-projection matrix to the shaderprogram */
        modelmatrix = modelmatrix * *projectionmatrix;
        glUniformMatrix4fv(glGetUniformLocation(program, "mvpMatrix"), 1, GL_FALSE, modelmatrix.data);

        glDrawElements(GL_TRIANGLES, bufferSize[index], GL_UNSIGNED_INT, 0);
    }
}

So my question now is: am I doing the right thing and if not: how do you guys suggest I should send the matrix data to the shader?

Many thanks in advance ;)

Sponsor:

#2 FXACE   Members   -  Reputation: 182

Like
0Likes
Like

Posted 23 August 2011 - 04:16 PM

I suppose using build-in OpenGL uniform variables is much better one ("mvMatrix" -> "gl_ModelViewMatrix", "normMatrix" - > "gl_NormalMatrix",...).where "gl_ModelViewMatrix" holds GL_MODELVIEW matrix state and so on 






#3 anonymous22   Members   -  Reputation: 199

Like
0Likes
Like

Posted 23 August 2011 - 04:56 PM

I suppose using build-in OpenGL uniform variables is much better one ("mvMatrix" -> "gl_ModelViewMatrix", "normMatrix" - > "gl_NormalMatrix",...).where "gl_ModelViewMatrix" holds GL_MODELVIEW matrix state and so on


Those built-ins are deprecated in 4.2+....

#4 karwosts   Members   -  Reputation: 840

Like
1Likes
Like

Posted 23 August 2011 - 09:58 PM

So what I did is declare the matrices in the shader as uniforms and made a neat updateMatrices() function for my DisplayObject class that is called everytime a transformation is done to the object (for example when it translates or rotates).


You're probably running into the problem of two objects using the same shader. If you use more than one object with a given shader, then you need to update the matrices each time you change between one and the other.

After some time of debugging I was able to fix this by calling chop the glUniform calls out of updateMatrices and call just before every object is drawn instead. Even though this works, I'm really doubting if I did the right thing here, because what I'm now doing is sending matrix data to the shader even though the values didn't change.

This sounds like a reasonable thing to do to me. Obviously you don't want to send data that you don't have to, but I can't imagine many cases where a single object gets a shader all to itself. I think just updating the matrices before drawing is not a big deal, unless you have unusual use case where it becomes impractical.
My Projects:
Portfolio Map for Android - Free Visual Portfolio Tracker
Electron Flux for Android - Free Puzzle/Logic Game

#5 krienie   Members   -  Reputation: 102

Like
0Likes
Like

Posted 24 August 2011 - 04:23 AM

...
This sounds like a reasonable thing to do to me. Obviously you don't want to send data that you don't have to, but I can't imagine many cases where a single object gets a shader all to itself. I think just updating the matrices before drawing is not a big deal, unless you have unusual use case where it becomes impractical.


That's what I thought. It seems a bit stupid to have two objects have it's own shader, each doing the exactly the same thing.
I'll leave it the way it is then. Thanks for the replies everyone ;)

#6 V-man   Members   -  Reputation: 797

Like
0Likes
Like

Posted 24 August 2011 - 09:45 AM

Don't call glGetUniformLocation. It is probably slow.
Sig: http://glhlib.sourceforge.net
an open source GLU replacement library. Much more modern than GLU.
float matrix[16], inverse_matrix[16];
glhLoadIdentityf2(matrix);
glhTranslatef2(matrix, 0.0, 0.0, 5.0);
glhRotateAboutXf2(matrix, angleInRadians);
glhScalef2(matrix, 1.0, 1.0, -1.0);
glhQuickInvertMatrixf2(matrix, inverse_matrix);
glUniformMatrix4fv(uniformLocation1, 1, FALSE, matrix);
glUniformMatrix4fv(uniformLocation2, 1, FALSE, inverse_matrix);

#7 Murdocki   Members   -  Reputation: 274

Like
1Likes
Like

Posted 24 August 2011 - 12:19 PM

You probably misinterpreted the "not changing frequently" part, uniforms are described as not changing frequently because they're the same for the whole draw call unlike vertex attributes which change every vertex. Also in reply to v-man calling glGetUnitformLocation is a call you are suggested to do load time rather than run time, you want to profile this on your target platform though, in my case it's lots faster to just make the calls than to map them.

#8 FXACE   Members   -  Reputation: 182

Like
0Likes
Like

Posted 24 August 2011 - 03:31 PM


I suppose using build-in OpenGL uniform variables is much better one ("mvMatrix" -> "gl_ModelViewMatrix", "normMatrix" - > "gl_NormalMatrix",...).where "gl_ModelViewMatrix" holds GL_MODELVIEW matrix state and so on


Those built-ins are deprecated in 4.2+....

There are many things deprecated in newest versions from classic GLSL. I'm old man if I don't know what's new in latest version of GLSL... 




Also, I'm agree with V-Man.



#9 phantom   Moderators   -  Reputation: 6801

Like
1Likes
Like

Posted 24 August 2011 - 05:18 PM

You probably misinterpreted the "not changing frequently" part, uniforms are described as not changing frequently because they're the same for the whole draw call unlike vertex attributes which change every vertex.


The term 'not changing frequently' applies just as well to something you set once during all your draw calls in a scene as it would to per-object.

In fact if your MVP matrix is only set once and never update during a scene/frame render setting it multiple times per scene/frame is wasteful and something which should be avoided (CPU time and GPU time overhead).

Now, I'm not saying there IS a way to use GLSL like that, although I hope there is because if not then that's a *facepalm* right there, but there really should be a way of doing it. My GLSL is just too rusty right now to point out that method if it exists :)

#10 V-man   Members   -  Reputation: 797

Like
1Likes
Like

Posted 25 August 2011 - 10:11 AM


You probably misinterpreted the "not changing frequently" part, uniforms are described as not changing frequently because they're the same for the whole draw call unlike vertex attributes which change every vertex.


The term 'not changing frequently' applies just as well to something you set once during all your draw calls in a scene as it would to per-object.

In fact if your MVP matrix is only set once and never update during a scene/frame render setting it multiple times per scene/frame is wasteful and something which should be avoided (CPU time and GPU time overhead).

Now, I'm not saying there IS a way to use GLSL like that, although I hope there is because if not then that's a *facepalm* right there, but there really should be a way of doing it. My GLSL is just too rusty right now to point out that method if it exists :)


Of course there is. UBO = Uniform buffer object
Which is basically a VBO for uniform values.
You need to make a GL 3.1 context to use it. Just shuv a GL_UNIFORM_BUFFER into your VBO function calls and get the location from your shader object.
http://www.opengl.or...entation/specs/

and you update the whole fucking uniform block inluding your MVP matrix in one go.
Sig: http://glhlib.sourceforge.net
an open source GLU replacement library. Much more modern than GLU.
float matrix[16], inverse_matrix[16];
glhLoadIdentityf2(matrix);
glhTranslatef2(matrix, 0.0, 0.0, 5.0);
glhRotateAboutXf2(matrix, angleInRadians);
glhScalef2(matrix, 1.0, 1.0, -1.0);
glhQuickInvertMatrixf2(matrix, inverse_matrix);
glUniformMatrix4fv(uniformLocation1, 1, FALSE, matrix);
glUniformMatrix4fv(uniformLocation2, 1, FALSE, inverse_matrix);

#11 krienie   Members   -  Reputation: 102

Like
0Likes
Like

Posted 25 August 2011 - 03:06 PM



You probably misinterpreted the "not changing frequently" part, uniforms are described as not changing frequently because they're the same for the whole draw call unlike vertex attributes which change every vertex.


The term 'not changing frequently' applies just as well to something you set once during all your draw calls in a scene as it would to per-object.

In fact if your MVP matrix is only set once and never update during a scene/frame render setting it multiple times per scene/frame is wasteful and something which should be avoided (CPU time and GPU time overhead).

Now, I'm not saying there IS a way to use GLSL like that, although I hope there is because if not then that's a *facepalm* right there, but there really should be a way of doing it. My GLSL is just too rusty right now to point out that method if it exists :)


Of course there is. UBO = Uniform buffer object
Which is basically a VBO for uniform values.
You need to make a GL 3.1 context to use it. Just shuv a GL_UNIFORM_BUFFER into your VBO function calls and get the location from your shader object.
http://www.opengl.or...entation/specs/

and you update the whole fucking uniform block inluding your MVP matrix in one go.



Now this is interesting! I'm totally going to check this out. Thank you very much! :)






Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.



PARTNERS