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.

Sign up for GitHub

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

Jump to bottom

Load culled tiles with low priorty #4969

Merged
merged 6 commits into from
Jan 29, 2018
Merged

Load culled tiles with low priorty #4969

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b1db1d6
Load culled tiles in case their OBB is wrong.
kring Feb 9, 2017
8167c85
Update CHANGES.md.
kring Feb 9, 2017
b51a502
Merge branch 'master' into missingTiles
Jan 11, 2018
00f164f
Update CHANGES.md
Jan 11, 2018
853910a
Merge branch 'master' into missingTiles
Jan 24, 2018
56ed3be
Merge branch 'master' into missingTiles
ggetz Jan 29, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion CHANGES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ Change Log
* Fixed a glTF animation bug that caused certain animations to jitter. [#5740](https://github.com/AnalyticalGraphicsInc/cesium/pull/5740)
* Fixed a bug when creating billboard and model entities without a globe. [#6109](https://github.com/AnalyticalGraphicsInc/cesium/pull/6109)
* Added support for vertex shader uniforms when `tileset.colorBlendMode` is `MIX` or `REPLACE`. [#5874](https://github.com/AnalyticalGraphicsInc/cesium/pull/5874)
* Added `shouldAnimate` option to the `Viewer` constructor to indicate if the clock should begin animating immediately.[#6154](https://github.com/AnalyticalGraphicsInc/cesium/pull/6154)
* Fixed a bug that could cause tiles to be missing from the globe surface, especially when starting with the camera zoomed close to the surface.
* Added `shouldAnimate` option to the `Viewer` constructor to indicate if the clock should begin animating immediately. [#6154](https://github.com/AnalyticalGraphicsInc/cesium/pull/6154)
* Added separate file for the Cesium [Code of Conduct](https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CODE_OF_CONDUCT.md). [#6129](https://github.com/AnalyticalGraphicsInc/cesium/pull/6129)
* Fixed applying a translucent style to a point cloud tileset. [#6113](https://github.com/AnalyticalGraphicsInc/cesium/pull/6113)
* Fixed sandcastle Particle System example for better visual [#6132](https://github.com/AnalyticalGraphicsInc/cesium/pull/6132)
Expand Down
9 changes: 8 additions & 1 deletion Source/Scene/QuadtreePrimitive.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ define([
var occluders = levelZeroTiles.length > 1 ? primitive._occluders : undefined;

// Sort the level zero tiles by the distance from the center to the camera.
// The level zero tiles aren't necessarily a nice neat quad, so we can use the
// The level zero tiles aren't necessarily a nice neat quad, so we can't use the
// quadtree ordering we use elsewhere in the tree
comparisonPoint = frameState.camera.positionCartographic;
levelZeroTiles.sort(compareDistanceToPoint);
Expand Down Expand Up @@ -653,6 +653,13 @@ define([
} else {
++primitive._debug.tilesCulled;
primitive._tileReplacementQueue.markTileRendered(tile);

// We've decided this tile is not visible, but if it's not fully loaded yet, we've made
// this determination based on possibly-incorrect information. We need to load this
// culled tile with low priority just in case it turns out to be visible after all.
if (tile.needsLoading) {
primitive._tileLoadQueueLow.push(tile);
}
}
}
/**
Expand Down