-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
3d-tiles branch cleanup #3177
Comments
One option for merging the various features is to avoid the hassle of cherry-picking and just open all of of the various PR's as planned (you can make each PR a single commit). Then, before opening a final PR for the 3d-tiles branch itself, we do a |
Great work @lilleyse! |
Given that the 3d-tile branch is starting to get big and is starting to get pull requests from many different developers, it needs to be brought up to production, and stable parts should be merged into master.
This looks like a lot, but I suspect it is less than a week's work.
Queue
/QueueSpec
(and related change to README.md); they are not used anymore.BillboardCollection
/Camera.distanceToBoundingSphere
changes or remove them. If we fix them, add tests and open a pull request into master.GlobeSurfaceTile
/GlobeSurfaceTileProvider
to useTileBoundingBox
. Add tests. Open a pull request into master.OrientedBoundingBox.fromCorners
and PR into master, e-andersson@e1f8903#commitcomment-15013420 (not needed - usingOrientedBoundingBox.fromRectangle
instead)git cherry-pick
when possible)Texture
/TextureSpec
(andGlobeSurfaceTile
) changes into master.RectangleGeometry
doc fixes into master. 79b22f4getStringFromTypedArray
changes into master.CullingVolume.computeVisibilityWithPlaneMask
/CullingVolumeSpec
into master.ModelAnimationCollection
into master. Add test if needed for coverage. 6a29ebaPerspectiveFrustum
andQuadtreePrimitive
into master. Add tests.Cartographic.fromCartesian
and open a pull request into master and3d-tiles
(TODO
in Cities.html).Primitive
/PrimitiveSpec
into master. Add tests.getBasePath
/Model
into master. Add tests. Use it anywhere else it is useful.Model
changes into master. (See me for how to diff this file between 3d-tiles and master).name
/id
renames.incrementallyLoadTextures
. Add doc and tests.precreatedAttributes
/vertexShaderLoaded
/fragmentShaderLoaded
/uniformMapLoaded
/pickVertexShaderLoaded
/pickFragmentShaderLoaded
/pickUniformMapLoaded
/ignoreCommands
. Add tests. No doc yet in case we need to change them.cull
. Add tests.PointGeometry
/PointAppearance
/PointAppearanceFS
/PointAppearanceVS
/createPointGeometry
to master. Document it as@private
and write tests.Cesium3DTileset
_statistics
andupdate
spys to check culling, tiles rendered, tiles requested, refined/didn't-need-to-refine, etc.content
defined (e.g., interior tiles in a quadtree)contents.box
definedBatched3DModel3DTileContentProvider
/BatchedModel
b3dm
tile with a batch table.b3dm
tile without a batch table.Instanced3DModel3DTileContentProvider
i3dm
tile with a batch table.i3dm
tile without a batch table.gltfFormat
options.ModelInstanceCollection
andModelInstance
- add lower-level explicit tests for these even though thei3dm
tests will cover them.Cesium3DTileBatchTableResources
- add explicit tests for this too.Points3DTileContentProvider
- tiles.json with onepnts
tile.Composite3DTileContentProvider
b3dm
andi3dm
now.update
for test assertions.Empty3DTileContentProvider
(trivial to test)Cesium3DTileContentProvider
(trivial to test)JobScheduler
yetWe'll replace the
DOC_TBA
comments in the 3D Tiles classes with actually reference doc later.The text was updated successfully, but these errors were encountered: