Sean Lilley

Results 189 issues of Sean Lilley

This PR adds support for the `3DTILES_content_gltf` extension which allows a tileset to refer to glTF and glb content directly. There's still backwards compatible support for tilesets with version `'2.0.0-alpha.0'`...

In https://github.com/CesiumGS/3d-tiles/issues/446 @renjianqiang pointed out that [batchColors.b3dm](https://github.com/CesiumGS/cesium/blob/master/Apps/SampleData/Cesium3DTiles/Batched/BatchedColors/batchedColors.b3dm) has a byte length of 13316 which is not 8-byte aligned as required by the Batched 3D Model format [here](https://github.com/CesiumGS/3d-tiles/tree/master/specification/TileFormats/Batched3DModel#padding). This and any...

bug

URIs in tileset JSON should be validated against https://tools.ietf.org/html/rfc3986#section-2.1. This means a tile uri like `My+tile.b3dm` is not valid. Instead it should be `My%2Btile.b3dm`

Now that KTX2/Basis support is being added to CesiumJS in https://github.com/CesiumGS/cesium/pull/9040 we should add a sample that uses KTX2/Basis textures. @YoussefV Take a look at `createBatchedTextured` as a reference point...

Instead of trying to maintain the legacy samples generator and 3D Tiles Next samples generator in the same project they should be separate folders.

cleanup

Tentative changes for `1.0` that should be handled in the `upgrade` command * Move batch table hierarchy to extension: https://github.com/AnalyticalGraphicsInc/3d-tiles/issues/303 * Fix leading slash: https://github.com/AnalyticalGraphicsInc/3d-tiles-tools/issues/141 * CESIUM_RTC to RTC_CENTER: https://github.com/AnalyticalGraphicsInc/3d-tiles-tools/issues/142...

Any glTFs that contain the [CESIUM_RTC](https://github.com/KhronosGroup/glTF/tree/master/extensions/1.0/Vendor/CESIUM_RTC) extension can instead us the `RTC_CENTER` feature table property: https://github.com/AnalyticalGraphicsInc/3d-tiles/issues/263.

If a tile url has a leading slash it should be treated as a warning, if not an error. See https://github.com/AnalyticalGraphicsInc/cesium/pull/6125

@rms13 `validateTileset` is doing a much better job now, but there are still a lot of loose ends. In the code verify that all required properties are present and are...

@rms13 Spec: https://github.com/AnalyticalGraphicsInc/3d-tiles/tree/master/TileFormats/BatchTable#batch-table-hierarchy Sandcastle demo in Cesium: http://cesiumjs.org/Cesium/Apps/Sandcastle/index.html?src=3D%20Tiles%20Batch%20Table%20Hierarchy.html&label=3D%20Tiles This is one of the more complicated areas of the 3D Tiles spec, so let me know if you have questions. Also...