Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Sampler States via Spritebatch not being applied #847

Closed
Upperfoot opened this Issue Oct 7, 2012 · 6 comments

Comments

Projects
None yet
5 participants

On current revision it seems like setting the sampler state has been broken, I am using this code to set the samplerstate for the spritebatch, but it seems that it is not being applied. Tried all sampler states and

spriteBatch.Begin(ssm, BlendState.AlphaBlend, Camera.SamplerState,
                        DepthStencilState.None, RasterizerState.CullCounterClockwise, null, view);

The commit I am currently using (where it is broken) is 4d9514e

The working commit for sampler states seems working on acda02b, but I have not narrowed it down to which commit it was actually broken on.

Any help on the issue?

Owner

tomspilman commented Oct 8, 2012

What platform are you seeing this issue on?

Contributor

elisee commented Oct 8, 2012

Possibly relevant pull request (#768) and bug report (#842) for reference.

It was on the windows platform I believe.

Contributor

shilrobot commented Oct 19, 2012

I posted a comment in bug #842 explaining what I believe is wrong and a possible fix -- basically the GL sampler state modification code is broken. I think both this bug and bug #842 have the same underlying cause.

@tomspilman tomspilman added a commit that referenced this issue Oct 26, 2012

@tomspilman tomspilman Merge pull request #889 from shilrobot/sampler-state-fix
Fix for issue #847, "SamplerStates aren't properly applied when active texture changes"
0ee113a
Contributor

shilrobot commented Oct 26, 2012

@Upperfoot can take a look and see if latest develop3d fixes this for you?

Contributor

danzel commented Apr 20, 2014

This works correctly on current Windows GL and DX. Close please

@tomspilman tomspilman closed this Apr 20, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment