embeds blendtotal into the low bit for the normal|depth buffer #1774

Merged
merged 1 commit into from Sep 23, 2016

Conversation

Projects
None yet
3 participants
@Azaezel
Contributor

Azaezel commented Sep 15, 2016

for terrains to support multi-pass blending.

Do note this will only address #1701 . Comprable blending artifacts for normals on objects with multiple layers with the normal maps on layer 0 and subsequent layers lacking a normal will need to be addressed separately.

@Areloch Areloch added the Bugfix label Sep 15, 2016

@Lopuska Lopuska added this to the 4.0 milestone Sep 17, 2016

@Lopuska Lopuska self-assigned this Sep 17, 2016

- //if ( prePassMat )
- //desc.setColorWrites( true, true, false, false );
+ if ( prePassMat )
+ desc.setColorWrites( true, true, true, false );

This comment has been minimized.

@Lopuska

Lopuska Sep 17, 2016

Member

Does color writes on prepass target was the responsible of the green shift we had long time ago on decals while working on fully deferred branch?
(Just to be sure before merge)

@Lopuska

Lopuska Sep 17, 2016

Member

Does color writes on prepass target was the responsible of the green shift we had long time ago on decals while working on fully deferred branch?
(Just to be sure before merge)

This comment has been minimized.

@Azaezel

Azaezel Sep 23, 2016

Contributor

yes. why we keep the blue channel around now regardless.

@Azaezel

Azaezel Sep 23, 2016

Contributor

yes. why we keep the blue channel around now regardless.

This comment has been minimized.

@Lopuska

Lopuska Sep 23, 2016

Member

great!

@Lopuska

Lopuska Sep 23, 2016

Member

great!

@Lopuska Lopuska merged commit 57dfeb8 into GarageGames:development Sep 23, 2016

@Azaezel Azaezel deleted the Azaezel:terrainTrouble branch Oct 19, 2016

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