-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
ModelExperimental
Sandcastle Testing
#10379
Comments
Today I looked into why the glTF 1.0 BIM model crashes in ModelExperimental. There's one shader that declares the normals as Trying to allow a special case for this in Some thoughts about how to update the model:
Unfortunately this upgrade process would have to be done on a case-by-case basis since glTF techniques are very open-ended (depends highly on the GLSL shader used) |
Came up in #10388: |
ModelExperimental Sandcastles
As part of #10346, I did a sweep through the old 3D Tiles and Model Sandcastles to see what happens when I set
enableModelExperimental = true
or useModelExperimental.fromGltf
instead ofModel.fromGltf
.EDITED: 07/08/2022
3D Tiles Sandcastles
Working Sandcastles:
Broken Sandcastles:
3D Tiles Batch Table Hierarchy-- fixed in Add some missing private methods inPropertyTable
for batch table hierarchy #10525batchTable.isExactClass
is not a functionbatchTable.isClass
is not a function3D Tiles Comparefixed in SupportCESIUM_primitive_outline
inModelExperimental
#105063D Tiles Feature Stylingfixed in Fix crash in 3D Tiles Feature Styling forModelExperimental
#10514ModelExperimental
#105143D Tiles Formatsfixed in Fix composite tile caching inModelExperimental
#10524enableDebugWireframe: true
3D Tiles Inspectorfixed in AddenableDebugWireframe
to 3D Tiles sandcastles #10526enableDebugWireframe: true
3D Tiles Interiorfixed in Fix composite tile caching inModelExperimental
#10524undefined
property valueDevelopment/3D Tiles Splitfixed in Fix composite tile caching inModelExperimental
#10524.otherwise()
Other 3D Tiles Sandcastles
These Sandcastles were found by searching "Cesium3DTileset" and "osmBuildings"
in the Sandcastle searchbar
Working Sandcastles:
Broken Sandcastles:
Cesium OSM Buildings- fixed in SupportCESIUM_primitive_outline
inModelExperimental
#10506Cloudsfixed in SupportCESIUM_primitive_outline
inModelExperimental
#10506Model Sandcastles
For these ones, I replaced
Model.fromGltf
withModelExperimental.fromGltf
Working Sandcastles:
Broken Sandcastles:
adjustment to the new IBL API
assets.agi.com
nodesByName
is undefinedThe text was updated successfully, but these errors were encountered: