Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transparency merge #618

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Transparency merge #618

wants to merge 46 commits into from

Conversation

psieg
Copy link
Contributor

@psieg psieg commented May 21, 2014

Adds a bitmask to hide the faces of adjacent voxels, which hopefully improves the visuals of transparent objects.
Also removes "inner" voxels from the renderdata avoiding the performance drop in the planet (not with the normandy's trail though)

psieg and others added 30 commits April 20, 2014 19:45
Conflicts:
	data/voxels.ini
	src/voxel/voxel.h
	src/voxel/voxelrenderdata.cpp
	src/voxeleffect/voxelparticleengine.cpp
	src/voxeleffect/voxelparticleengine.h
	src/voxeleffect/voxelparticlerenderer.h
	src/voxeleffect/voxelparticlesetup.h
@mrzzzrm
Copy link

mrzzzrm commented May 22, 2014

Shall this be reviewed separate from/after #589, or can #589 be closed?

@psieg
Copy link
Contributor Author

psieg commented May 22, 2014

It contains #589, I'd consider it a dependency (review after transparency).

gl_Position = viewProjection * model * (vec4(v_position.xyz + v_vertex.xyz * 0.999, 1.0));

f_normal = (model * vec4(v_normal.xyz, 0.0)).xyz;
if ((uint(v_faces) & (uint(0x1) << uint(v_normal.w))) != uint(0x0)) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indendation

psieg added 2 commits May 31, 2014 18:46
Conflicts:
	data/shader/postprocessing/combine.frag
	data/shader/voxelcluster/voxelcluster.frag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants