LÖVE

kikito wrote:The sprites look like random color noise

   vec3 light; // <-- Could be any value at this point.
   for (int i = 0; i < light_position.length(); i++) {      
      vec3 light_direction = light_position[i] - vec3(pixel_coords, 0);
      float dist = length(light_direction);
      float attenuation = smoothstep(100, 50, dist);
      light_direction = normalize(light_direction);
      vec3 current_light = light_diffuse[i] * attenuation * clamp(dot(normal, light_direction), 0.0, 1.0);
      //light += light_diffuse[i] * attenuation * clamp(dot(normal, light_direction), 0.0, 1.0);
      if (length(current_light) > length(light)) { // <-- Usage of uninitialized variable. If light has random bits, there is a very high chance this will always be false.
         light = current_light;
      }
   }

kikito wrote:Unfortunately none of the multi-lighted examples seem to work on my machine (Mac Book Air 2012 13')

The sprites look like random color noise:

The first two examples (with only one light source) looked great though.

Boolsheet wrote:

kikito wrote:The sprites look like random color noise

This is just a guess, but it may be because the shader uses a uninitialized variable.

Code: Select all

   vec3 light; // <-- Could be any value at this point.
   for (int i = 0; i < light_position.length(); i++) {      
      vec3 light_direction = light_position[i] - vec3(pixel_coords, 0);
      float dist = length(light_direction);
      float attenuation = smoothstep(100, 50, dist);
      light_direction = normalize(light_direction);
      vec3 current_light = light_diffuse[i] * attenuation * clamp(dot(normal, light_direction), 0.0, 1.0);
      //light += light_diffuse[i] * attenuation * clamp(dot(normal, light_direction), 0.0, 1.0);
      if (length(current_light) > length(light)) { // <-- Usage of uninitialized variable. If light has random bits, there is a very high chance this will always be false.
         light = current_light;
      }
   }

Nixola wrote:

Code: Select all

Error: [string "graphics.lua"]:1346: Invalid operation:
- Trying to send the wrong value type to shader variable, or
- Trying to send array values with wrong dimension, or
- Invalid variable name.
stack traceback:
        [C]: in function 'sendFloat'
        [string "graphics.lua"]:1346: in function 'send'
        main.lua:20: in function 'send_lights'
        main.lua:89: in function 'update'
        [string "boot.lua"]:407: in function <[string "boot.lua"]:373>
        [C]: in function 'xpcall'

Intel G41 Express Chipset, Windows 7 x86


EDIT: Fixed, you were sending two 5-element arrays when the shader expected two 4-element ones, just change lines 2-3 to

Code: Select all

extern vec3 light_position[5]; //it was [4]
extern vec3 light_diffuse[5]; //it was [4]