Discussion :

[zenux]

Bonjour,

J'ai un fragment shader en GLSL qui ne compile pas.
L'erreur qui en ressort est assez inhabituelle et elle ne m'aide pas vraiment à trouver le problème:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
#version 330
#extension GL_EXT_gpu_shader4 : enable
 
in vec2 textCoordinates;
 
//deferred textures:
uniform sampler2D depthTex;
uniform sampler2D colorTex;
uniform sampler2D normalAndAmbientTex;
uniform mat4 mInverseViewProjection;
 
//lights and shadows:
struct StructLightInfo
{
	bool isExist;
	vec3 position;
	bool produceShadow;
 
	float constantAtt;
	float linearAtt;
	float quadraticAtt;
	vec4 lightAmbient;
 
	sampler2DArray depthShadowTex;
	mat4 mLightProjectionView[3];
	mat4 mShadowOffset[3];
};
uniform StructLightInfo lightsInfo[16];
uniform float depthSplitDistance[3];
uniform vec4 globalAmbient;
 
out vec4 fragColor;
 
void main()
{
	vec4 diffuse = texture2D(colorTex, textCoordinates);
	vec4 normalAndAmbient = vec4(texture2D(normalAndAmbientTex, textCoordinates));
	vec3 normal = vec3(normalAndAmbient) * 2.0 - 1.0;
	float modelAmbientFactor = normalAndAmbient.a;
 
	vec4 texPosition = vec4(
		textCoordinates.s * 2.0 - 1.0,
		textCoordinates.t * 2.0 - 1.0,
		texture2D(depthTex, textCoordinates).r * 2.0 - 1.0,
		1.0
	);
	vec4 position = mInverseViewProjection * texPosition;
	position /= position.w;
 
	if(modelAmbientFactor >= 0.999)
	{ //no lighting
		fragColor = diffuse;
	}else
	{
		fragColor = globalAmbient * modelAmbientFactor;
		for(int i=0; i<16;++i)
		{
			if(lightsInfo[i].isExist)
			{
				vec3 vertexToLight = lightsInfo[i].position - vec3(position);
				vec3 vertexToLightNormalized = normalize(vertexToLight);
 
				float NdotL = max(dot(normal, vertexToLightNormalized), 0.0);
 
				fragColor += NdotL; //diffuse * NdotL;
			}
		}
	}
}

Si je commente la dernière ligne (fragColor), je n'ai plus de problème. Il semble que le problème se produit quand j'utilise la variable NdotL.

Voici l'erreur:

Fragment info
-------------
Internal error: assembly compile error for fragment shader at offset 15355:
-- error message --
line 250, column 35: error: invalid local parameter number
line 256, column 19: error: out of bounds array access
line 258, column 18: error: out of bounds array access
line 259, column 35: error: out of bounds array access
line 263, column 18: error: out of bounds array access
line 268, column 17: error: out of bounds array access
//........
-- internal assembly text --
!!NVfp4.1
OPTION NV_parameter_buffer_object2;
# cgc version 3.1.0001, build date Apr 30 2012
# command line args:
#vendor NVIDIA Corporation
#version 3.1.0.1
#profile gp4_1fp
#program main
#semantic depthTex
#semantic colorTex
#semantic normalAndAmbientTex
#semantic mInverseViewProjection
#semantic lightsInfo
//.......

Si nécessaire, je peux vous donner le message d'erreur complet mais il est assez long.

Merci d'avance.

[pyros]

Salut,

Et en replaçant par fragColor = fragColor + vec4(NdotL) ?
Les compilo GLSL sont parfois capricieux sur les conversions implicites ou les expressions un peu trop compacte.

Mais d'après l'erreur (out of bounds array access), ça ressemble à un problème avec lightsInfo[i]. Essaye de replacer lightsInfo[i].isExist par true et lightsInfo[i].position par vec3(1) par ex. Je ne sais pas si les tableaux de structure de tableaux sont supportés en GLSL 3.3 mais même s'ils le sont, on est pas à l'abris d'un bug de compilo...

[zenux]

Quand je diminue mon nombre d'uniform, je n'ai plus d'erreur.

Je viens de regarder la limite d'uniform (GL_MAX_FRAGMENT_UNIFORM_COMPONENTS) et j'obtiens 2048.
Quand je fais le calcul pour mon shader, j'obtiens ceci: 1+1+1+16+((1+3+1+1+1+1+4+1+16*3+16*3)*16)+3*1+4 = 1770.
Je suis donc en dessous de 2048.

Par contre, je ne sais pas combien vaut un sampler2D et un sampler2DArray (j'ai compté 1). Une idée ?

[dragonjoker59]

Je pense que passer 16 fois ta structure pose un gros soucis.
Je me souviens que je ne pouvais pas passer plus de 4 lumières (position, couleur, attenuations, cutoff et exponent) sur certaines architectures, raison pour laquelle je suis passé aux textures pour passer mes lumières (je peux maintenant sans problème en passer 100)

[LittleWhite]

Bonjour,

En effet sur toutes les cartes graphiques, nous avons des limites sur le nombres d'uniforms et de données que l'on peut passer au shader.
Comme il a été dit, la solution qui suit, c'est d'utiliser les textures.

Avec glGet() nous pouvons connaitre le nombre de variables pouvant être passées

[zenux]

Merci pour vos réponses.
Je vais donc passer aux textures.