• Advertisement
Sign in to follow this  

Deferred Shading - strange results

This topic is 1891 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

Im trying to get a simple deferred shading working.
I have GBuffer with 3 textures ( image, position, normal ).
Image :
Normal :
I'm trying to compute point light ( when i was used directional light i almost work ) like that :
// Fragment Shader //
#version 330 core
out vec4 outputColor;
in block
vec2 TexCoord;
vec3 Normal;
} In;
uniform sampler2D gDiffuseSampler;
uniform sampler2D gPositionSampler;
uniform sampler2D gNormalSampler;
uniform vec3 lightPosition;
uniform vec3 lightColor;
uniform float lightRadius;
void main()
vec4 image = texture( gDiffuseSampler, In.TexCoord );
vec4 position = texture( gPositionSampler, In.TexCoord );
vec4 normal = texture( gNormalSampler, In.TexCoord );

vec3 lightDir = lightPosition - position.xyz ;

normal = normalize(normal);
lightDir = normalize(lightDir);

float att = clamp(1.0f-length(lightDir)/lightRadius,0.0,1.0);
float diff = max(dot(normal,lightDir), 0);

outputColor = att * vec4(diff * lightColor,1.0);
i'm binding it on fullscreen quad.
Result :
It is looking like "camera is center of point light ;| dafuq? i write about ten different shaders ;| on every the same - "camera is point light center", if you set camera closer to object, light get stronger ;o? i dont know, what im doing wrong. Sorry for my bad english.
I will by grateful for your help.

Share this post

Link to post
Share on other sites
Are you using floating point or integer textures. Do not forget that the latter can only hold values in range 0.0...1.0
Thus if you are reading (and storing) your position (and normal) values from texture you have to transform these:

vec4 outputcolor = 0.5 * (vec4(1.0, 1.0, 1.0, 1.0) + position);

vec4 position = 2.0 * (vec4(0.5, 0.5, 0.5, 0.5) + texture( gPositionSampler, In.TexCoord ));

Share this post

Link to post
Share on other sites
Sign in to follow this  

  • Advertisement