Skip to content

Commit

Permalink
Hide symbols behind the camera
Browse files Browse the repository at this point in the history
  • Loading branch information
mpulkki-mapbox committed Dec 10, 2019
1 parent 4f78874 commit 167e4a5
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/shaders/symbol_icon.vertex.glsl
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,10 @@ void main() {
vec4 projected_pos = u_label_plane_matrix * vec4(a_projected_pos.xy, 0.0, 1.0);
gl_Position = u_coord_matrix * vec4(projected_pos.xy / projected_pos.w + rotation_matrix * (a_offset / 32.0 * max(a_minFontScale, fontScale) + a_pxoffset / 16.0), 0.0, 1.0);

// Depth value is always set to zero even for points behind the camera.
// Disable rendering of these points by setting their depth to be < -1.0
gl_Position.z = mix(-2.0 * gl_Position.w, 0.0, float(projected_pos.w > 0.0));

v_tex = a_tex / u_texsize;
vec2 fade_opacity = unpack_opacity(a_fade_opacity);
float fade_change = fade_opacity[1] > 0.5 ? u_fade_change : -u_fade_change;
Expand Down
4 changes: 4 additions & 0 deletions src/shaders/symbol_sdf.vertex.glsl
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,10 @@ void main() {
gl_Position = u_coord_matrix * vec4(projected_pos.xy / projected_pos.w + rotation_matrix * (a_offset / 32.0 * fontScale + a_pxoffset), 0.0, 1.0);
float gamma_scale = gl_Position.w;

// Depth value is always set to zero even for points behind the camera.
// Disable rendering of these points by setting their depth to be < -1.0
gl_Position.z = mix(-2.0 * gl_Position.w, 0.0, float(projected_pos.w > 0.0));

vec2 fade_opacity = unpack_opacity(a_fade_opacity);
float fade_change = fade_opacity[1] > 0.5 ? u_fade_change : -u_fade_change;
float interpolated_fade_opacity = max(0.0, min(1.0, fade_opacity[0] + fade_change));
Expand Down
10 changes: 9 additions & 1 deletion src/symbol/projection.js
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,15 @@ function updateLineLabels(bucket: SymbolBucket,
fontSize / perspectiveRatio;

const tileAnchorPoint = new Point(symbol.anchorX, symbol.anchorY);
const anchorPoint = project(tileAnchorPoint, labelPlaneMatrix).point;
const transformedTileAnchor = project(tileAnchorPoint, labelPlaneMatrix);

// Skip labels behind the camera
if (transformedTileAnchor.signedDistanceFromCamera <= 0.0) {
hideGlyphs(symbol.numGlyphs, dynamicLayoutVertexArray);
continue;
}

const anchorPoint = transformedTileAnchor.point;
const projectionCache = {};

const placeUnflipped: any = placeGlyphsAlongLine(symbol, pitchScaledFontSize, false /*unflipped*/, keepUpright, posMatrix, labelPlaneMatrix, glCoordMatrix,
Expand Down

0 comments on commit 167e4a5

Please sign in to comment.