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.

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

Move Gltf-Pipeline to Source/Scene #9699

Merged
merged 4 commits into from
Jul 30, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
1 change: 1 addition & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ Apps/HelloWorld.html
Apps/Sandcastle/ThirdParty/**
Build/**
Documentation/**
Source/Scene/GltfPipeline/**
Source/Shaders/**
Source/ThirdParty/**
Source/ThirdPartyNpm/**
Expand Down
1 change: 1 addition & 0 deletions .prettierignore
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
# Re-ignore a few things caught above
**/*.min.js
Source/Cesium.js
Source/Scene/GltfPipeline/**
Source/Shaders/**/*.js
Source/ThirdParty/**
Source/ThirdPartyNpm/**
Expand Down
12 changes: 6 additions & 6 deletions Source/Scene/ClassificationModel.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,12 @@ import PrimitiveType from "../Core/PrimitiveType.js";
import RuntimeError from "../Core/RuntimeError.js";
import Transforms from "../Core/Transforms.js";
import WebGLConstants from "../Core/WebGLConstants.js";
import addDefaults from "../ThirdParty/GltfPipeline/addDefaults.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import getAccessorByteStride from "../ThirdParty/GltfPipeline/getAccessorByteStride.js";
import numberOfComponentsForType from "../ThirdParty/GltfPipeline/numberOfComponentsForType.js";
import parseGlb from "../ThirdParty/GltfPipeline/parseGlb.js";
import updateVersion from "../ThirdParty/GltfPipeline/updateVersion.js";
import addDefaults from "./GltfPipeline/addDefaults.js";
import ForEach from "./GltfPipeline/ForEach.js";
import getAccessorByteStride from "./GltfPipeline/getAccessorByteStride.js";
import numberOfComponentsForType from "./GltfPipeline/numberOfComponentsForType.js";
import parseGlb from "./GltfPipeline/parseGlb.js";
import updateVersion from "./GltfPipeline/updateVersion.js";
import when from "../ThirdPartyNpm/when.js";
import Axis from "./Axis.js";
import ModelLoadResources from "./ModelLoadResources.js";
Expand Down
2 changes: 1 addition & 1 deletion Source/Scene/DracoLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import ComponentDatatype from "../Core/ComponentDatatype.js";
import defined from "../Core/defined.js";
import FeatureDetection from "../Core/FeatureDetection.js";
import TaskProcessor from "../Core/TaskProcessor.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import ForEach from "./GltfPipeline/ForEach.js";
import when from "../ThirdPartyNpm/when.js";

/**
Expand Down
12 changes: 6 additions & 6 deletions Source/Scene/GltfJsonLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@ import getJsonFromTypedArray from "../Core/getJsonFromTypedArray.js";
import getMagic from "../Core/getMagic.js";
import isDataUri from "../Core/isDataUri.js";
import Resource from "../Core/Resource.js";
import addDefaults from "../ThirdParty/GltfPipeline/addDefaults.js";
import addPipelineExtras from "../ThirdParty/GltfPipeline/addPipelineExtras.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import parseGlb from "../ThirdParty/GltfPipeline/parseGlb.js";
import removePipelineExtras from "../ThirdParty/GltfPipeline/removePipelineExtras.js";
import updateVersion from "../ThirdParty/GltfPipeline/updateVersion.js";
import addDefaults from "./GltfPipeline/addDefaults.js";
import addPipelineExtras from "./GltfPipeline/addPipelineExtras.js";
import ForEach from "./GltfPipeline/ForEach.js";
import parseGlb from "./GltfPipeline/parseGlb.js";
import removePipelineExtras from "./GltfPipeline/removePipelineExtras.js";
import updateVersion from "./GltfPipeline/updateVersion.js";
import when from "../ThirdPartyNpm/when.js";
import ResourceLoader from "./ResourceLoader.js";
import ResourceLoaderState from "./ResourceLoaderState.js";
Expand Down
6 changes: 3 additions & 3 deletions Source/Scene/GltfLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@ import FeatureDetection from "../Core/FeatureDetection.js";
import Matrix4 from "../Core/Matrix4.js";
import Quaternion from "../Core/Quaternion.js";
import Sampler from "../Renderer/Sampler.js";
import getAccessorByteStride from "../ThirdParty/GltfPipeline/getAccessorByteStride.js";
import getComponentReader from "../ThirdParty/GltfPipeline/getComponentReader.js";
import numberOfComponentsForType from "../ThirdParty/GltfPipeline/numberOfComponentsForType.js";
import getAccessorByteStride from "./GltfPipeline/getAccessorByteStride.js";
import getComponentReader from "./GltfPipeline/getComponentReader.js";
import numberOfComponentsForType from "./GltfPipeline/numberOfComponentsForType.js";
import when from "../ThirdPartyNpm/when.js";
import AttributeType from "./AttributeType.js";
import GltfFeatureMetadataLoader from "./GltfFeatureMetadataLoader.js";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ import defined from "../../Core/defined.js"
* Mimics the behavior of functions in gltf-pipeline ForEach.
* @param {Object} material The glTF material.
* @param {forEachTextureInMaterial~handler} handler Function that is called for each texture in the material.
*
* @private
*/
function forEachTextureInMaterial(material, handler) {
Check.typeOf.object('material', material);
Expand Down Expand Up @@ -129,6 +131,8 @@ function forEachTextureInMaterial(material, handler) {
* @callback forEachTextureInMaterial~handler
* @param {Number} The texture index.
* @param {Object} The texture info object.
*
* @private
*/

export default forEachTextureInMaterial;
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,8 @@ function getComponentReader(componentType) {
* @param {Number} numberOfComponents The number of components to read.
* @param {Number} componentTypeByteLength The byte length of each component.
* @param {Number} result An array storing the components that are read.
*
* @private
*/

export default getComponentReader;
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,10 @@ Remove.buffer = function(gltf, bufferId) {
if (defined(bufferView.buffer) && bufferView.buffer > bufferId) {
bufferView.buffer--;
}

if (defined(bufferView.extensions) && defined(bufferView.extensions.EXT_meshopt_compression)) {
bufferView.extensions.EXT_meshopt_compression.buffer--;
}
});
};

Expand Down Expand Up @@ -424,6 +428,9 @@ getListOfElementsIdsInUse.buffer = function(gltf) {
if (defined(bufferView.buffer)) {
usedBufferIds[bufferView.buffer] = true;
}
if (defined(bufferView.extensions) && defined(bufferView.extensions.EXT_meshopt_compression)) {
usedBufferIds[bufferView.extensions.EXT_meshopt_compression.buffer] = true;
}
});

return usedBufferIds;
Expand Down
16 changes: 8 additions & 8 deletions Source/Scene/Model.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,14 +43,14 @@ import Texture from "../Renderer/Texture.js";
import TextureMinificationFilter from "../Renderer/TextureMinificationFilter.js";
import TextureWrap from "../Renderer/TextureWrap.js";
import VertexArray from "../Renderer/VertexArray.js";
import addDefaults from "../ThirdParty/GltfPipeline/addDefaults.js";
import addPipelineExtras from "../ThirdParty/GltfPipeline/addPipelineExtras.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import getAccessorByteStride from "../ThirdParty/GltfPipeline/getAccessorByteStride.js";
import usesExtension from "../ThirdParty/GltfPipeline/usesExtension.js";
import numberOfComponentsForType from "../ThirdParty/GltfPipeline/numberOfComponentsForType.js";
import parseGlb from "../ThirdParty/GltfPipeline/parseGlb.js";
import updateVersion from "../ThirdParty/GltfPipeline/updateVersion.js";
import addDefaults from "./GltfPipeline/addDefaults.js";
import addPipelineExtras from "./GltfPipeline/addPipelineExtras.js";
import ForEach from "./GltfPipeline/ForEach.js";
import getAccessorByteStride from "./GltfPipeline/getAccessorByteStride.js";
import usesExtension from "./GltfPipeline/usesExtension.js";
import numberOfComponentsForType from "./GltfPipeline/numberOfComponentsForType.js";
import parseGlb from "./GltfPipeline/parseGlb.js";
import updateVersion from "./GltfPipeline/updateVersion.js";
import when from "../ThirdPartyNpm/when.js";
import Axis from "./Axis.js";
import BlendingState from "./BlendingState.js";
Expand Down
4 changes: 2 additions & 2 deletions Source/Scene/ModelAnimationCache.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ import QuaternionSpline from "../Core/QuaternionSpline.js";
import Spline from "../Core/Spline.js";
import WebGLConstants from "../Core/WebGLConstants.js";
import WeightSpline from "../Core/WeightSpline.js";
import getAccessorByteStride from "../ThirdParty/GltfPipeline/getAccessorByteStride.js";
import numberOfComponentsForType from "../ThirdParty/GltfPipeline/numberOfComponentsForType.js";
import getAccessorByteStride from "./GltfPipeline/getAccessorByteStride.js";
import numberOfComponentsForType from "./GltfPipeline/numberOfComponentsForType.js";
import AttributeType from "./AttributeType.js";

/**
Expand Down
2 changes: 1 addition & 1 deletion Source/Scene/ModelInstanceCollection.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ import DrawCommand from "../Renderer/DrawCommand.js";
import Pass from "../Renderer/Pass.js";
import RenderState from "../Renderer/RenderState.js";
import ShaderSource from "../Renderer/ShaderSource.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import ForEach from "./GltfPipeline/ForEach.js";
import when from "../ThirdPartyNpm/when.js";
import Model from "./Model.js";
import ModelInstance from "./ModelInstance.js";
Expand Down
2 changes: 1 addition & 1 deletion Source/Scene/ModelOutlineLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import Texture from "../Renderer/Texture.js";
import TextureMagnificationFilter from "../Renderer/TextureMagnificationFilter.js";
import TextureMinificationFilter from "../Renderer/TextureMinificationFilter.js";
import TextureWrap from "../Renderer/TextureWrap.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import ForEach from "./GltfPipeline/ForEach.js";

// glTF does not allow an index value of 65535 because this is the primitive
// restart value in some APIs.
Expand Down
6 changes: 3 additions & 3 deletions Source/Scene/ModelUtility.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ import Quaternion from "../Core/Quaternion.js";
import RuntimeError from "../Core/RuntimeError.js";
import WebGLConstants from "../Core/WebGLConstants.js";
import ShaderSource from "../Renderer/ShaderSource.js";
import addToArray from "../ThirdParty/GltfPipeline/addToArray.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import usesExtension from "../ThirdParty/GltfPipeline/usesExtension.js";
import addToArray from "./GltfPipeline/addToArray.js";
import ForEach from "./GltfPipeline/ForEach.js";
import usesExtension from "./GltfPipeline/usesExtension.js";
import AttributeType from "./AttributeType.js";
import Axis from "./Axis.js";

Expand Down
6 changes: 3 additions & 3 deletions Source/Scene/processModelMaterialsCommon.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ import defaultValue from "../Core/defaultValue.js";
import defined from "../Core/defined.js";
import WebGLConstants from "../Core/WebGLConstants.js";
import webGLConstantToGlslType from "../Core/webGLConstantToGlslType.js";
import addToArray from "../ThirdParty/GltfPipeline/addToArray.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import usesExtension from "../ThirdParty/GltfPipeline/usesExtension.js";
import addToArray from "./GltfPipeline/addToArray.js";
import ForEach from "./GltfPipeline/ForEach.js";
import usesExtension from "./GltfPipeline/usesExtension.js";
import ModelUtility from "./ModelUtility.js";

/**
Expand Down
6 changes: 3 additions & 3 deletions Source/Scene/processPbrMaterials.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ import defaultValue from "../Core/defaultValue.js";
import defined from "../Core/defined.js";
import WebGLConstants from "../Core/WebGLConstants.js";
import webGLConstantToGlslType from "../Core/webGLConstantToGlslType.js";
import addToArray from "../ThirdParty/GltfPipeline/addToArray.js";
import ForEach from "../ThirdParty/GltfPipeline/ForEach.js";
import usesExtension from "../ThirdParty/GltfPipeline/usesExtension.js";
import addToArray from "./GltfPipeline/addToArray.js";
import ForEach from "./GltfPipeline/ForEach.js";
import usesExtension from "./GltfPipeline/usesExtension.js";
import ModelUtility from "./ModelUtility.js";

/**
Expand Down
2 changes: 1 addition & 1 deletion Specs/Scene/GltfBuilder.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { defined, RuntimeError } from "../../Source/Cesium.js";
import findAccessorMinMax from "../../Source/ThirdParty/GltfPipeline/findAccessorMinMax.js";
import findAccessorMinMax from "../../Source/Scene/GltfPipeline/findAccessorMinMax.js";

/**
* A fluent interface for programmatically building a glTF.
Expand Down