black textures about luminance-rs HOT 9 CLOSED

Yeah, it’s on me. I’ll patch that soon. I won’t close your issue so that I can use it as backlog, if you don’t mind. The patch is not hard to write, it just requires a bit of refactoring, nothing hard. I’ll do that on this weekend if I find enough spare time.

Thanks again for reaching out!

from luminance-rs.

I don’t have renderdoc right now. I’ll have a look when I get home. What sampler are you using in your shader?

from luminance-rs.

Sampler {
    wrap_r: texture::Wrap::Repeat, 
    wrap_s: texture::Wrap::Repeat,
    wrap_t: texture::Wrap::Repeat,
    min_filter: texture::MinFilter::Linear,
    mag_filter: texture::MagFilter::Linear,
    depth_comparison: None,
};

in vec2 fPos;

out vec4 color;

uniform sampler2D tex;

void main() {

    color = vec4(texture(tex, fPos).rgb, 1.0);
    //color = vec4(1.0, 1.0, 1.0, clamp(fPos.x + fPos.y, 0.0, 1.0));
}

from luminance-rs.

Try with uniform usampler2D tex;, because your texture has an internal format of RGBA8UI.

On a more general note, I need to patch the check code for this. Most uniforms would have yielded a type mismatch (you can try it out), but with arrays and textures, I just assume it’s always ok for now.

from luminance-rs.

yep, that was it! thanks a bunch. i checked for all uniform warnings, so i didn't think my uniform types could have been off after i didn't get any more warnings! good to know

from luminance-rs.

OK, I’ll work on that right now.

from luminance-rs.

@tinaun I’ll release a new version of luminance soon but if you want to test with the current HEAD, you might give it a try! Just use the initial code you posted. You should now have a type mismatch.

from luminance-rs.

awesome!

from luminance-rs.

luminance-0.28 ships with it! :)

from luminance-rs.