Sign in to follow this  
tom_mai78101

OpenGL I am no longer able to load a cube mesh correctly. How do I fix this problem?

Recommended Posts

tom_mai78101    693

Wall of text below:

 

I'm having trouble loading up a correct cube mesh using OpenGL ES 2.0 on an Android device running ICS.

 

From what I observed, I see that the vertices of the cube is not only distorted, but also some of the vertices have incorrect vertex positions in the world matrix. You can tell by looking at some of the snapshots taken below:

 

vCO2fbl.pngWdD7AcX.pngW8MaGV9.png

 

 

 

 

 

I have checked and confirmed that my OBJ file parser is not causing any troubles, by comparing the values copied into the buffer objects to the values in the OBJ file itself. I have used the software, Blender, to re-create the cube mesh by exporting and overwriting the OBJ files. I have exclusively enabled "Triangulate faces" and "Include normal" in the Blender -> Export options. None of the actions above worked.

 

I have also checked my shader codes to see if they compiled and are referencing the correct variables. So far, there's nothing wrong. The OpenGL ES did not spit out any errors, when I was using glGetProgramInfo() and glGetShaderInfo(). They all pass. And I can see my model clearly, just that I didn't add any ambient lighting and shadows to the mesh.

 

I have somehow recreated the problem that I had asked a month ago prior that was solved by fixing the transformation matrix calculation methods. But this time, it's no longer the matrices. I have checked and confirmed that the transformation matrices are not causing the problem.

 

The cube that was exported/overwritten is the same cube mesh I had used in the Stack Overflow question above. Which looked like this below: (Left side: Perspective view of the back side of the cube mesh. Right side: Orthogonal view of the front size of the cube mesh.)

 

l0EaZpW.png

 

Below is an attachment of the Android APK, packed in a ZIP file.

 

[attachment=14673:3DOpenGL.zip]

 

Below is the project source file attached:

 

[attachment=14674:3DOpenGL_source.zip]

 

And below are some of the source code that I used to load the cube mesh model:  (Model class, OBJ model loader static method) They are shown for easier lookup.

 

package models;

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.nio.ByteBuffer;
import java.nio.ByteOrder;
import java.nio.FloatBuffer;
import java.nio.ShortBuffer;
import java.util.ArrayList;
import android.content.Context;
import android.content.res.AssetManager;
import android.util.Log;

public class Model {
	
	protected FloatBuffer vertexBuffer;
	protected ShortBuffer indexBuffer;
	
	public static Model loadOBJModel(Context c, String filename) {
		
		ArrayList<Float> tempVertices = new ArrayList<Float>();
		ArrayList<Float> tempNormals = new ArrayList<Float>();
		ArrayList<Short> tempVertexIndices = new ArrayList<Short>();
		ArrayList<Short> tempNormalIndices = new ArrayList<Short>();
		Model model = new Model();
		
		try {
			AssetManager manager = c.getAssets();
			BufferedReader reader = new BufferedReader(new InputStreamReader(manager.open(filename)));
			String line;
			while ((line = reader.readLine()) != null) {
				if (line.startsWith("v")) {
					for (int i = 0; i < 3; i++) {
						tempVertices.add(Float.valueOf(line.split(" ")[i + 1]));
					}
				}
				else if (line.startsWith("vn")) {
					for (int i = 0; i < 3; i++) {
						tempNormals.add(Float.valueOf(line.split(" ")[i + 1]));
					}
				}
				else if (line.startsWith("f")) {
					String[] tokens = line.split(" ");
					if (tokens[1].contains("/")) {
						for (int i = 0; i < 3; i++) {
							tempVertexIndices.add(Short.valueOf(tokens[i + 1].split("/")[0]));
							tempNormalIndices.add(Short.valueOf(tokens[i + 1].split("/")[2]));
						}
					}
					else {
						for (int i = 0; i < 3; i++) {
							tempVertexIndices.add(Short.valueOf(tokens[i + 1]));
						}
					}
				}
			}
			
			
			//TODO: Will add lighting back once everything is confirmed to work properly.
			float[] vertices = createFloatArray(tempVertices);
			//float[] normals= createFloatArray(tempNormals);
			short[] vertexIndices = createShortArray(tempVertexIndices);
			//short[] normalIndices = createShortArray(tempNormalIndices);
			
			model.vertexBuffer = ByteBuffer.allocateDirect(vertices.length * 4).order(ByteOrder.nativeOrder()).asFloatBuffer();
			model.vertexBuffer.put(vertices).position(0);
			model.indexBuffer = ByteBuffer.allocateDirect(vertexIndices.length * 2).order(ByteOrder.nativeOrder()).asShortBuffer();
			model.indexBuffer.put(vertexIndices).position(0);
			
		}
		catch (Exception e) {
			e.printStackTrace();
			Log.e("DEBUG loadCube()", "Exception error caught.", e);
		}
		
		return model;
	}
	
	//--------------------------------------
	
	private static float[] createFloatArray(ArrayList<Float> list) {
		float[] array = new float[list.size()];
		for (int i = 0; i < array.length; i++) {
			Float f = list.get(i);
			array[i] = (f != null ? f : Float.NaN);
		}
		return array;
	}
	
	private static short[] createShortArray(ArrayList<Short> list) {
		short[] array = new short[list.size()];
		for (int i = 0; i < array.length; i++) {
			Short f = list.get(i);
			array[i] = (f != null ? f : 0);
		}
		return array;
	}
}

 

Below is the Cube class: (Cube class, with transformation matrices, vertex and fragment shader codes, and the initialization)

 

package models;

import java.util.Arrays;
import android.content.Context;
import android.opengl.GLES20;
import android.opengl.Matrix;
import android.util.Log;


public class Cube {
	private String vertexCode = "" +
			"attribute vec4 a_position;              	\n" +
			"uniform mat4 u_mvpMatrix;                	\n" +
			"void main() {                           	\n" +
			"	gl_Position = u_mvpMatrix * a_position;	\n" +
			"}                         					\n";
	
	private String fragmentCode = "" +
			"precision mediump float;                   \n" +
			"void main() {                              \n" +
			"	gl_FragColor = vec4(0.0, 1.0, 0.0, 1.0);\n" +
			"}                                          \n";
	
	private int shaderProgram;
	
	private int attribute_Position;
	private int uniform_mvpMatrix;
	
	private float angle;
	private Model model;
	
	public Cube(Context c, String filename){
		model = Model.loadOBJModel(c, filename);
	}
	
	
	public void setup(){
		int vertexShader = GLES20.glCreateShader(GLES20.GL_VERTEX_SHADER);
		GLES20.glShaderSource(vertexShader, vertexCode);
		GLES20.glCompileShader(vertexShader);
		int fragmentShader = GLES20.glCreateShader(GLES20.GL_FRAGMENT_SHADER);
		GLES20.glShaderSource(fragmentShader, fragmentCode);
		GLES20.glCompileShader(fragmentShader);
		
		shaderProgram = GLES20.glCreateProgram();
		GLES20.glAttachShader(shaderProgram, vertexShader);
		GLES20.glAttachShader(shaderProgram, fragmentShader);
		GLES20.glBindAttribLocation(shaderProgram, 1, "a_position");
		GLES20.glLinkProgram(shaderProgram);
		
		int[] linked = new int[1];
		GLES20.glGetProgramiv(shaderProgram, GLES20.GL_LINK_STATUS, linked, 0);
		if (linked[0] == 0) {
			Log.e("DEBUG setup()", "Shader code error.");
			Log.e("DEBUG setup()", GLES20.glGetProgramInfoLog(shaderProgram));
			GLES20.glDeleteProgram(shaderProgram);
			return;
		}
		
		GLES20.glDeleteShader(vertexShader);
		GLES20.glDeleteShader(fragmentShader);
		
		attribute_Position = GLES20.glGetAttribLocation(shaderProgram, "a_position");
		uniform_mvpMatrix = GLES20.glGetUniformLocation(shaderProgram, "u_mvpMatrix");
		GLES20.glEnableVertexAttribArray(attribute_Position);
	}
	
	public void draw(float[] mMatrix, float[] vMatrix, float[] pMatrix){
		float[] mvpMatrix = new float[16];
		Matrix.setIdentityM(mMatrix, 0);
		Matrix.translateM(mMatrix, 0, 0f, -2.5f, -1.5f);
		Matrix.rotateM(mMatrix, 0, angle, 0f, 1f, 0f);
		
		// This is the transformation matrices that I had a problem with a month ago. It has been solved by using
		// an array copy of the original matrix. The behavior is already defined in the Android documentation.
		
		Matrix.multiplyMM(mvpMatrix, 0, vMatrix, 0, mMatrix, 0);
		Matrix.multiplyMM(mvpMatrix, 0, pMatrix, 0, Arrays.copyOf(mvpMatrix, 16), 0);
		
		
		
		GLES20.glUseProgram(shaderProgram);
		GLES20.glVertexAttribPointer(attribute_Position, 3, GLES20.GL_FLOAT, false, 3 * 4, model.vertexBuffer);
		
		GLES20.glUniformMatrix4fv(uniform_mvpMatrix, 1, false, mvpMatrix, 0);
		
		GLES20.glDrawElements(GLES20.GL_TRIANGLES, model.indexBuffer.capacity(), GLES20.GL_UNSIGNED_SHORT, model.indexBuffer);
		
		angle = ++angle % 360f;
	}
}

 

Oh boy, there's a lot of information provided just so I can nail this bug, squish it, and move on. I really need help on this one. Been looking for answers on Google search, and came up dry. The first three results are not what I wanted. The second result is the link that I asked for solutions a month ago. The rest of the results are tutorials.

 

If there's something I might have missed. Please let me know.

Edited by tom_mai78101

Share this post


Link to post
Share on other sites
Sponji    2503

There are at least two bugs in your model loading code:

 

line.startsWith("v")

What happens to lines with which start with "vn"? "vn" startsWith "v". This doesn't really matter right now, but you get unnecessary vertices, which are actually normals.

 

tempVertexIndices.add(Short.valueOf(tokens[i + 1].split("/")[0]));

You have to subtract one from the index numbers, because obj format starts counting indices from 1 instead of 0.

 

Hope this helps.

Edited by Sponji

Share this post


Link to post
Share on other sites
tom_mai78101    693

I'm currently debugging this problem, along with your tips.

line.startsWith("v");

 can also take in "v", "vn", and possibly "vt"? I did not know that...

 

--------------------------------------

 

Another question that I'm more confused is when the index buffer is read, does the index numbers in the buffers have to start off like:

 

0, 1, 2, 3, 4,...

 

And not the other way around?

 

1, 2, 3, 4, 5,...

 

I thought the index number and the vertex element in OpenGL ES reads the same, and that OpenGL ES automatically chooses the index element position accordingly. I'm checking this as I go.

Share this post


Link to post
Share on other sites
L. Spiro    25638

That is not what he means.

The values of the indices inside the .OBJ file are 1 greater than they should be (I seriously don’t understand the purpose of this—shame on the creators of .OBJ for what seems to be a newbie trap and nothing more), which means in order to make it useful to your program you need to subtract 1 from that value, not add one.

 

 

L. Spiro

Share this post


Link to post
Share on other sites
tom_mai78101    693

Thanks again for your input. I've now fully understood what all of you were saying.

 

I finally got the free time to continue debugging, and it looks like using:

String.startsWith(String prefix);

Is a bad idea. I've changed it to doing:

 

String header = line_read_from_input.split(" ")[0];
header.equals(String prefix);

So it's always correct.

 

Everything is now perfect. Life is good. Thanks to all and all a good night.

Edited by tom_mai78101

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  

  • Partner Spotlight

  • Similar Content

    • By pseudomarvin
      I assumed that if a shader is computationally expensive then the execution is just slower. But running the following GLSL FS instead just crashes
      void main() { float x = 0; float y = 0; int sum = 0; for (float x = 0; x < 10; x += 0.00005) { for (float y = 0; y < 10; y += 0.00005) { sum++; } } fragColor = vec4(1, 1, 1 , 1.0); } with unhandled exception in nvoglv32.dll. Are there any hard limits on the number of steps/time that a shader can take before it is shut down? I was thinking about implementing some time intensive computation in shaders where it would take on the order of seconds to compute a frame, is that possible? Thanks.
    • By Arulbabu Donbosco
      There are studios selling applications which is just copying any 3Dgraphic content and regenerating into another new window. especially for CAVE Virtual reality experience. so that the user opens REvite or CAD or any other 3D applications and opens a model. then when the user selects the rendered window the VR application copies the 3D model information from the OpenGL window. 
      I got the clue that the VR application replaces the windows opengl32.dll file. how this is possible ... how can we copy the 3d content from the current OpenGL window.
      anyone, please help me .. how to go further... to create an application like VR CAVE. 
       
      Thanks
    • By cebugdev
      hi all,

      i am trying to build an OpenGL 2D GUI system, (yeah yeah, i know i should not be re inventing the wheel, but this is for educational and some other purpose only),
      i have built GUI system before using 2D systems such as that of HTML/JS canvas, but in 2D system, i can directly match a mouse coordinates to the actual graphic coordinates with additional computation for screen size/ratio/scale ofcourse.
      now i want to port it to OpenGL, i know that to render a 2D object in OpenGL we specify coordiantes in Clip space or use the orthographic projection, now heres what i need help about.
      1. what is the right way of rendering the GUI? is it thru drawing in clip space or switching to ortho projection?
      2. from screen coordinates (top left is 0,0 nd bottom right is width height), how can i map the mouse coordinates to OpenGL 2D so that mouse events such as button click works? In consideration ofcourse to the current screen/size dimension.
      3. when let say if the screen size/dimension is different, how to handle this? in my previous javascript 2D engine using canvas, i just have my working coordinates and then just perform the bitblk or copying my working canvas to screen canvas and scale the mouse coordinates from there, in OpenGL how to work on a multiple screen sizes (more like an OpenGL ES question).
      lastly, if you guys know any books, resources, links or tutorials that handle or discuss this, i found one with marekknows opengl game engine website but its not free,
      Just let me know. Did not have any luck finding resource in google for writing our own OpenGL GUI framework.
      IF there are no any available online, just let me know, what things do i need to look into for OpenGL and i will study them one by one to make it work.
      thank you, and looking forward to positive replies.
    • By fllwr0491
      I have a few beginner questions about tesselation that I really have no clue.
      The opengl wiki doesn't seem to talk anything about the details.
       
      What is the relationship between TCS layout out and TES layout in?
      How does the tesselator know how control points are organized?
          e.g. If TES input requests triangles, but TCS can output N vertices.
             What happens in this case?
      In this article,
      http://www.informit.com/articles/article.aspx?p=2120983
      the isoline example TCS out=4, but TES in=isoline.
      And gl_TessCoord is only a single one.
      So which ones are the control points?
      How are tesselator building primitives?
    • By Orella
      I've been developing a 2D Engine using SFML + ImGui.
      Here you can see an image
      The editor is rendered using ImGui and the scene window is a sf::RenderTexture where I draw the GameObjects and then is converted to ImGui::Image to render it in the editor.
      Now I need to create a 3D Engine during this year in my Bachelor Degree but using SDL2 + ImGui and I want to recreate what I did with the 2D Engine. 
      I've managed to render the editor like I did in the 2D Engine using this example that comes with ImGui. 
      3D Editor preview
      But I don't know how to create an equivalent of sf::RenderTexture in SDL2, so I can draw the 3D scene there and convert it to ImGui::Image to show it in the editor.
      If you can provide code will be better. And if you want me to provide any specific code tell me.
      Thanks!
  • Popular Now