Sign in to follow this  
Ahl

VBO help...

Recommended Posts

Ahl    170
Having trouble getting my VBOs to work. No compiler errors. No run time errors. Just nothing showing up on the screen. VA's work though. I'm wondering if my implementation is the issue. I'll try to explain as best I can...

All the model information gets stored in the following:

[CODE]
struct MyVertex
{
float x, y, z; // Vertex
float nx, ny, nz; // Normal
float u, v; // Texcoords
float a, r, g, b; // Color
float padding[4];
};
[/CODE]

This is how I load the VBO:

[CODE]
for (Itter = Mesh_Multimap.begin(); Itter != Mesh_Multimap.end(); ++Itter)
{
VBO_Size += (*Itter).second->NumVerticies*sizeof(MyVertex);
}
glGenBuffersARB( 1, &VBO_ID );
glBindBufferARB( GL_ARRAY_BUFFER_ARB, VBO_ID );
glBufferDataARB( GL_ARRAY_BUFFER_ARB, VBO_Size, NULL, GL_STATIC_DRAW_ARB );
for (VBO_Vector_IT = VBO_Vector.begin(); VBO_Vector_IT != VBO_Vector.end(); ++VBO_Vector_IT)
{
glBufferSubDataARB( GL_ARRAY_BUFFER_ARB, VBO_Offset, (*VBO_Vector_IT)->NumVerticies*sizeof(MyVertex), (*VBO_Vector_IT)->VertexData );
(*VBO_Vector_IT)->VBO_Offset = VBO_Offset;
VBO_Offset += (*VBO_Vector_IT)->NumVerticies*sizeof(MyVertex);
}
[/CODE]

Rendering:

[CODE]
glVertexPointer( 3, GL_FLOAT, 0, &(*RenderListIT)->RenderMesh->VBO_Offset );
glNormalPointer( GL_FLOAT, 0, &(*RenderListIT)->RenderMesh->VBO_Offset );
glTexCoordPointer( 2, GL_FLOAT, 0, &(*RenderListIT)->RenderMesh->VBO_Offset );
[/CODE]

That's the point that's giving me pause. When just rendering with VAs I need to do the following:

[CODE]
glVertexPointer( 3, GL_FLOAT, sizeof(MyVertex), &(*RenderListIT)->RenderMesh->VertexData->x );
glNormalPointer( GL_FLOAT, sizeof(MyVertex), &(*RenderListIT)->RenderMesh->VertexData->nx );
glTexCoordPointer( 2, GL_FLOAT, sizeof(MyVertex), &(*RenderListIT)->RenderMesh->VertexData->u );
glColorPointer( 3, GL_FLOAT, sizeof(MyVertex), &(*RenderListIT)->RenderMesh->VertexData->a );
[/CODE]

I had to specify what part of the struct to pull the appropriate data from. Am I supposed to be doing this with the VBOs somehow? What am I missing?

Share this post


Link to post
Share on other sites
Brother Bob    10344
Looks very wrong that you're passing the address of the offset variables to the gl*Pointer functions. If the offset variables actually contains the offsets into the VBOs, then you need to pass their values and not their addresses.

Share this post


Link to post
Share on other sites
Ahl    170
Ooh, good catch. Didn't even see that. I converted it over from where I needed it for the VAs and forgot about that. Thank you.

Stuff is showing up now but it's all scrambled...

Share this post


Link to post
Share on other sites
Ahl    170
Yeah, still getting garbled models, even when just loading one. I have this sneaky suspician that it has to do with not being able (or not know how) to specift what part of my struct each pointer needs to be using...

Share this post


Link to post
Share on other sites
Brother Bob    10344
It didn't occur to me first, but you have the same offset for all attributes. The vertex pointer needs the offset to the start of the vertex data, of course, and likewise for the other attribute arrays. Edited by Brother Bob

Share this post


Link to post
Share on other sites
Ahl    170
[CODE]
struct MyVertex
{
float x, y, z; // Vertex
float nx, ny, nz; // Normal
float u, v; // Texcoords
float a, r, g, b; // Color
float padding[4];
};
[/CODE]

Keep in mind that this is my data structure. An array of this is what is being loaded into the buffer, so the offset will be the same for all of the vertexs/normals/texcoords/etc per model.

I figured out another problem. I had the stride set to 0. I set it to sizeof(MyVertex) and now my model is coming up properly. The textures are still wonky though. It's doing now what it was doing before when we figured out I had to do the whole &(*RenderListIT)->RenderMesh->VertexData->x bit.

Share this post


Link to post
Share on other sites
BitMaster    8651
[quote name='Ahl' timestamp='1348017039' post='4981497']
[CODE]
struct MyVertex
{
float x, y, z; // Vertex
float nx, ny, nz; // Normal
float u, v; // Texcoords
float a, r, g, b; // Color
float padding[4];
};
[/CODE]

Keep in mind that this is my data structure. An array of this is what is being loaded into the buffer, so the offset will be the same for all of the vertexs/normals/texcoords/etc per model.
[/quote]

No. If vertex data starts at offset 0 (the most common case in that scenario), then normal data starts at offset 0 + sizeof(float) * 3 and so on. Otherwise your normals are always your positions of the same vertex, your texture coordinates are just your (x, y) coordinates of the same vertex and your colors are the (x, y, z, nx) coordinates of the same vertex.

Setting the stride to sizeof(MyVertex) was correct but only half the problem.

Share this post


Link to post
Share on other sites
mhagain    13430
You can do something like this:[code]MyVertex dummy = NULL;

glVertexPointer (3, GL_FLOAT, sizeof (MyVertex), &dummy->x);
glNormalPointer (GL_FLOAT, sizeof (MyVertex), &dummy->nx);
glTexCoordPointer (2, GL_FLOAT, sizeof (MyVertex), &dummy->u);
glColorPointer (3, GL_FLOAT, sizeof (MyVertex), &dummy->a);[/code]

It looks awful but it will set up the offsets correctly for you, doesn't require any casting of a number to (void *), allows you to modify your struct (e.g by adding another element - maybe a second set of texcoords - somewhere in the middle of it) without breaking your VBO setup, and makes it easier to co-exist code for regular vertex arrays with VBOs. Just be careful to not do anything else with "dummy" aside from this as it will crash your program.

Generally I'd use an array of xyz[3] instead of individual x, y, z but that's just a stylistic preference.

Share this post


Link to post
Share on other sites
Ahl    170
[quote name='BitMaster' timestamp='1348039369' post='4981577']
[quote name='Ahl' timestamp='1348017039' post='4981497']
[CODE]
struct MyVertex
{
float x, y, z; // Vertex
float nx, ny, nz; // Normal
float u, v; // Texcoords
float a, r, g, b; // Color
float padding[4];
};
[/CODE]

Keep in mind that this is my data structure. An array of this is what is being loaded into the buffer, so the offset will be the same for all of the vertexs/normals/texcoords/etc per model.
[/quote]

No. If vertex data starts at offset 0 (the most common case in that scenario), then normal data starts at offset 0 + sizeof(float) * 3 and so on. Otherwise your normals are always your positions of the same vertex, your texture coordinates are just your (x, y) coordinates of the same vertex and your colors are the (x, y, z, nx) coordinates of the same vertex.

Setting the stride to sizeof(MyVertex) was correct but only half the problem.
[/quote]

Boom! That did it. Thank you very much!

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this