Discussion :

[filoo]

Hello!

J'ai les trois std::vector suivant:

vertex, contenant les coordonnées (float x,y,z) des vertices
face, contenant les 3 indices (GLint) des vertices (structure a,b,c)
normal, contenant les normales (float x,y,z)
..

J'ai testé avec glDrawArray(GL_POINT, ...) en liant les pointeurs GL_VERTEX_ARRAY et GL_NORMAL array vers les std::vector correspondant et ça... ça marche bien.

Je voudrais maintenant afficher les face en utilisant le vector face comme liste d'indices...

avec glDrawElements(GL_TRIANGLES, face.size(), GL_INT, &face[0]);

mais ça, ça marche pas ... rien ne s'affiche.

Pourquoi?

[filoo]

Voilà le code de la fonction de dessin ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void Mesh::Draw(){
 
        glEnableClientState(GL_VERTEX_ARRAY);
        glEnableClientState(GL_NORMAL_ARRAY);
        glVertexPointer(3, GL_FLOAT, 0, &vertex[0]);
        glNormalPointer(GL_FLOAT, 0, &normal[0]);
        glColor4f(1.0,1.0,1.0,1.0);
 
        glDrawElements(GL_TRIANGLES, face.size(), GL_INT, &face[0]);
 
        glDisableClientState(GL_VERTEX_ARRAY);
        glDisableClientState(GL_NORMAL_ARRAY); }

[Laurent Gomila]

Faudrait pas aussi activer GL_INDEX_ARRAY avec glEnableClientState ?

[filoo]

J'ai testé, ça ne marche pas mieux...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
glEnableClientState(GL_VERTEX_ARRAY);
        glEnableClientState(GL_NORMAL_ARRAY);
        glEnableClientState(GL_INDEX_ARRAY);
        glVertexPointer(3, GL_FLOAT, 0, &vertex[0]);
        glNormalPointer(GL_FLOAT, 0, &normal[0]);
        glPointSize(2.0);
        glColor4f(1.0,1.0,1.0,1.0);
        //glDrawArrays(GL_POINTS, 0, vertex.size()); <-- ça, ça marche...
        glDrawElements(GL_TRIANGLES, sizeof(GLint)*3*face.size(), GL_INT, &face[0]);
        glDisableClientState(GL_VERTEX_ARRAY);
        glDisableClientState(GL_NORMAL_ARRAY);
        glDisableClientState(GL_INDEX_ARRAY);

[filoo]

J'ai testé avec la méthode classique:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
glBegin(GL_TRIANGLES);
        for(int i=0; i<face.size(); i++)
        {
          glNormal3f(normal.at(face.at(i).a).x, normal.at(face.at(i).a).y, normal.at(face.at(i).a).z);
          glVertex3f(vertex.at(face.at(i).a).x, vertex.at(face.at(i).a).y, vertex.at(face.at(i).a).z);
          glNormal3f(normal.at(face.at(i).b).x, normal.at(face.at(i).b).y, normal.at(face.at(i).b).z);
          glVertex3f(vertex.at(face.at(i).b).x, vertex.at(face.at(i).b).y, vertex.at(face.at(i).b).z);
          glNormal3f(normal.at(face.at(i).c).x, normal.at(face.at(i).c).y, normal.at(face.at(i).c).z);
          glVertex3f(vertex.at(face.at(i).c).x, vertex.at(face.at(i).c).y, vertex.at(face.at(i).c).z);
        }
       glEnd();

et ça marche.
Une idée pourquoi l'autre méthode ne marche pas?

[bafman]

pourquoi

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

sizeof(GLint)*3*face.size()

sachant que face est ton tableau d'indices, il n'y a pas besoin du 3 non ?

[filoo]

3 parce que le tableau est un vector de structure Face

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
struct Face{
GLint a; 
GLint b; 
GLint c;
}

donc 3 GLint par entrée du vecteur... d'où le x3.

[Bakura]

Voilà le code de la fonction de dessin ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void Mesh::Draw(){
 
        glEnableClientState(GL_VERTEX_ARRAY);
        glEnableClientState(GL_NORMAL_ARRAY);
        glVertexPointer(3, GL_FLOAT, 0, &vertex[0]);
        glNormalPointer(GL_FLOAT, 0, &normal[0]);
        glColor4f(1.0,1.0,1.0,1.0);
 
        glDrawElements(GL_TRIANGLES, face.size(), GL_INT, &face[0]);
 
        glDisableClientState(GL_VERTEX_ARRAY);
        glDisableClientState(GL_NORMAL_ARRAY); }

glDrawElements(GL_TRIANGLES, face.size(), GL_INT, &face[0]);

Qu'est-ce que face.size () renvoit ? Le nombre de faces ou le nombre de vertices ? Car c'est le nombre de vertices qui doit être mis ici, donc face.size() * 3, et ce n'est pas un pointeur vers le première élément de face qui doit être envoyé mais de vertex.

EDIT : j'ai supprimé la fin, je crois que j'ai dit des bêtises après :p

[filoo]

Auto-trouvé! Ca marche finalement ...

L'erreur était dans le type de données attribuées dans glDrawElement

à savoir qu'il devait être du type GL_UNSIGNED_INT et pas GL_INT.

Voilà le résultat.



PS: c'est bien un pointeur vers l'array d'index qu'il faut passer à la fin de l'appel à glDrawElement.

Merci du coup de main!

[smashy]

je trouve personnellement dangereux d utiliser un vector<face> et de faire comme si &v[0] te retournait un entier.

Effectivement, il y a de bonne chance que pour v[0] et le premier entier aient bien la meme addresse. En revanche, tu n est a priori pas sur que le compilateur ne rajoutra pas des octets superflux pour l alignement (padding) a la fin du 3eme indice ("c") dans ta structure.

question subsidiaire : est ce que quelqu'un sait si la consigne de compilation

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

#pragma pack est portable

[Laurent Gomila]

Aucun #pragma n'est portable, puisqu'ils servent par définition à implémenter des trucs spécifiques aux compilos.

Par contre j'ai un jour croisé des en-têtes pour ça :
<pshpack1.h>
<pshpack8.h>
...
<poppack.h>

Par contre je pense que ça ne fonctionne que sous Windows. Si tu veux un truc complétement portable, il faut te palucher les directives particulières de tous les compilos.

[smashy]

c est bien ce que je pensais ....

conclusion : utiliser un tableau de structures qui contiennent 3 int , n est pas equivalent a utiliser un tableau de int.

[Yno]

Faudrait pas aussi activer GL_INDEX_ARRAY avec glEnableClientState ?

Les tableaux d'indices ne s'activent pas. GL_INDEX_ARRAY représente les couleurs indexées (faudra corriger ça dans le Yes::Engine ).

« Les tableaux d'indices doivent êtres des tableaux d'entiers non signés. Et c'est important, sinon ça ne fonctionne pas »

EDIT :

à savoir qu'il devait être du type GL_UNSIGNED_INT et pas GL_INT.

Eh ben, oui

[Laurent Gomila]

Les tableaux d'indices ne s'activent pas. GL_INDEX_ARRAY représente les couleurs indexées (faudra corriger ça dans le Yes::Engine ).

Arg, je me fais toujours avoir avec ça
Ils auraient pu caser le mot COLOR dans leurs constantes désignant les couleurs indicées.

[bafman]

Arg, je me fais toujours avoir avec ça
Ils auraient pu caser le mot COLOR dans leurs constantes désignant les couleurs indicées.

c'est sûre que c'est piegeant comme truc, mais il faut savoir que les couleur indexé date d'openGL 1.0 alors que les vertex array date du 1.1, donc à l'epoque d'openGL 1.0, le seul truc qui pouvait être indexé, c'etait les couleurs