-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add tests #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 1
Configuration used: CodeRabbit UI
Files selected for processing (16)
- src/models/tileset.rs (1 hunks)
- tests/load_gltf.rs (1 hunks)
- tests/samples/gltf/CESIUM_primitive_outline/BoxPrimitiveOutline.gltf (1 hunks)
- tests/samples/gltf/EXT_mesh_features/FeatureIdAttribute.gltf (1 hunks)
- tests/samples/gltf/EXT_mesh_features/FeatureIdTexture.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/ComplexTypes.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/FeatureIdAttributeAndPropertyTable.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/FeatureIdTextureAndPropertyTable.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/MultipleClasses.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/MultipleFeatureIdsAndProperties.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/PropertyAttributesPointCloudHouse.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/PropertyAttributesPointCloudTree.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/SharedPropertyTable.gltf (1 hunks)
- tests/samples/gltf/EXT_structural_metadata/SimplePropertyTexture.gltf (1 hunks)
- tests/samples/gltf/GpuInstanceMetadata/GpuInstancesMetadata.gltf (1 hunks)
- tests/samples/gltf/README.md (1 hunks)
Additional comments: 29
tests/samples/gltf/README.md (1)
- 1-3: 内容に問題はありません。
tests/samples/gltf/EXT_structural_metadata/PropertyAttributesPointCloudTree.gltf (1)
- 1-115: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/EXT_structural_metadata/PropertyAttributesPointCloudHouse.gltf (1)
- 1-115: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/EXT_mesh_features/FeatureIdAttribute.gltf (1)
- 1-102: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/CESIUM_primitive_outline/BoxPrimitiveOutline.gltf (1)
- 1-103: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/EXT_structural_metadata/FeatureIdAttributeAndPropertyTable.gltf (1)
- 1-141: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/EXT_structural_metadata/MultipleFeatureIdsAndProperties.gltf (1)
- 1-177: GLTFサンプルファイルの内容に問題はありません。
tests/samples/gltf/GpuInstanceMetadata/GpuInstancesMetadata.gltf (5)
- 2-33: このGLTFファイルは、
EXT_structural_metadata
、EXT_mesh_gpu_instancing
、およびEXT_instance_features
の3つの拡張機能を使用しています。これらの拡張機能は、GPUインスタンシングと構造的メタデータのサポートを提供し、3Dモデルの表現を強化します。ファイルの構造と拡張機能の使用は適切に見えます。- 34-94: アクセサーの定義は、バッファビュー、バイトオフセット、コンポーネントタイプ、カウント、タイプ、最大値、および最小値を含む完全な情報を提供しています。これらのアクセサーは、モデルのジオメトリと属性データへのアクセスを可能にします。各アクセサーの定義は適切に見え、GLTF仕様に従っています。
- 95-104: バッファセクションは、モデルデータをエンコードしたBase64文字列を含むURIを提供します。これにより、GLTFファイルは外部ファイルへの依存なしに、すべての必要なデータを自己完結させることができます。バッファの定義は適切で、データのエンコーディングも正しく行われているように見えます。
- 105-144: バッファビューは、バッファ内の特定の部分への参照を提供し、アクセサーが使用するデータの範囲を定義します。ターゲットプロパティは、バッファビューが頂点属性データまたはインデックスデータのいずれに使用されるかを示します。バッファビューの定義は適切で、GLTF仕様に従っています。
- 145-178: メッシュとノードのセクションは、3Dモデルの構造を定義します。メッシュはプリミティブの配列を含み、各プリミティブはジオメトリとマテリアルの情報を持ちます。ノードはシーン内のオブジェクトの階層構造を表し、メッシュへの参照を含むことができます。これらのセクションの定義は適切で、3Dモデルの表現に必要な情報を提供しています。
tests/samples/gltf/EXT_mesh_features/FeatureIdTexture.gltf (6)
- 2-2: このGLTFファイルは、
EXT_mesh_features
拡張機能を使用しています。この拡張機能は、メッシュの特徴をより詳細に制御するためのものです。ファイルの構造と拡張機能の使用は適切に見えます。- 3-34: アクセサーの定義は、バッファビュー、バイトオフセット、コンポーネントタイプ、カウント、タイプ、最大値、および最小値を含む完全な情報を提供しています。これらのアクセサーは、モデルのジオメトリと属性データへのアクセスを可能にします。各アクセサーの定義は適切に見え、GLTF仕様に従っています。
- 36-38: アセットセクションは、GLTFファイルが生成されたツールの情報を提供します。この情報は、ファイルの互換性やデバッグに役立つ場合があります。定義は適切で、GLTF仕様に従っています。
- 40-42: バッファセクションは、モデルデータをエンコードしたBase64文字列を含むURIを提供します。これにより、GLTFファイルは外部ファイルへの依存なしに、すべての必要なデータを自己完結させることができます。バッファの定義は適切で、データのエンコーディングも正しく行われているように見えます。
- 44-63: バッファビューは、バッファ内の特定の部分への参照を提供し、アクセサーが使用するデータの範囲を定義します。ターゲットプロパティは、バッファビューが頂点属性データまたはインデックスデータのいずれに使用されるかを示します。バッファビューの定義は適切で、GLTF仕様に従っています。
- 65-126: イメージ、マテリアル、メッシュ、ノード、サンプラー、シーン、テクスチャのセクションは、3Dモデルのビジュアル表現に関連する情報を提供します。これらのセクションは、テクスチャの使用、マテリアルのプロパティ、メッシュの構造、およびシーンの構成を定義します。これらのセクションの定義は適切で、GLTF仕様に従っており、
EXT_mesh_features
拡張機能との統合も適切に行われています。tests/samples/gltf/EXT_structural_metadata/SharedPropertyTable.gltf (5)
- 2-33: このGLTFファイルは、
EXT_mesh_features
およびEXT_structural_metadata
の2つの拡張機能を使用しています。これらの拡張機能は、メッシュの特徴と構造的メタデータのサポートを提供し、3Dモデルの表現を強化します。ファイルの構造と拡張機能の使用は適切に見えます。- 34-97: アクセサーの定義は、バッファビュー、バイトオフセット、コンポーネントタイプ、カウント、タイプ、最大値、および最小値を含む完全な情報を提供しています。これらのアクセサーは、モデルのジオメトリと属性データへのアクセスを可能にします。各アクセサーの定義は適切に見え、GLTF仕様に従っています。
- 99-101: アセットセクションは、GLTFファイルが生成されたツールの情報を提供します。この情報は、ファイルの互換性やデバッグに役立つ場合があります。定義は適切で、GLTF仕様に従っています。
- 103-156: バッファとバッファビューのセクションは、モデルデータをエンコードしたBase64文字列を含むURIを提供し、アクセサーが使用するデータの範囲を定義します。これらのセクションの定義は適切で、データのエンコーディングも正しく行われているように見えます。バッファビューは、GLTF仕様に従っており、適切に構成されています。
- 157-219: マテリアル、メッシュ、ノード、シーンのセクションは、3Dモデルのビジュアル表現と構造を定義します。これらのセクションは、マテリアルのプロパティ、メッシュの構造、およびシーンの構成を定義し、
EXT_structural_metadata
拡張機能との統合を示します。これらのセクションの定義は適切で、GLTF仕様に従っており、3Dモデルの表現に必要な情報を提供しています。tests/samples/gltf/EXT_structural_metadata/ComplexTypes.gltf (1)
- 1-214: このGLTFファイルは、
EXT_structural_metadata
拡張機能を使用して複雑な型のメタデータを定義しています。メタデータクラス、プロパティ、列挙型、プロパティテーブルの定義が適切に行われており、GLTFフォーマットの規約に従っています。特に、可変長および固定長の配列、正規化された数値、ENUM型の使用が見られます。これらの定義は、GLTFファイル内での3Dモデルのメタデータ表現を豊かにするための良い例です。tests/samples/gltf/EXT_structural_metadata/MultipleClasses.gltf (1)
- 1-228: このGLTFファイルは、
EXT_structural_metadata
拡張機能を使用して複数のメタデータクラスを定義しています。各クラスは異なるプロパティを持ち、FLOAT32、INT64、UINT16、FLOAT64などの異なるコンポーネントタイプが使用されています。これにより、3Dモデルのメタデータをより詳細に表現することが可能になります。ファイルはGLTFフォーマットの規約に従っており、適切に構成されています。tests/samples/gltf/EXT_structural_metadata/FeatureIdTextureAndPropertyTable.gltf (1)
- 1-180: このGLTFファイルは、
EXT_structural_metadata
拡張機能を使用して、テクスチャを利用した特徴IDの定義とプロパティテーブルの組み合わせを示しています。建物のコンポーネントと建設年をメタデータとして持ち、3Dモデルに関連する追加情報を提供します。テクスチャを使用した特徴IDの定義は、3Dモデルの特定の部分にメタデータを関連付ける効果的な方法です。ファイルはGLTFフォーマットの規約に従っており、適切に構成されています。tests/samples/gltf/EXT_structural_metadata/SimplePropertyTexture.gltf (1)
- 1-170: このGLTFファイルは、
EXT_structural_metadata
拡張機能を使用して建物のコンポーネントを定義しています。このファイルは、内部温度、外部温度、断熱材の厚さなどのプロパティを持つ建物のコンポーネントのスキーマを定義しています。また、プロパティテクスチャ、アクセサ、アセット情報、バッファ、バッファビュー、イメージ、マテリアル、メッシュ、ノード、サンプラー、シーン、テクスチャを含んでいます。このファイルは、GLTFファイルの取り扱いとEXT_structural_metadata
拡張機能の使用方法に関する良い例です。src/models/tileset.rs (2)
- 437-441:
test_default
関数は、デフォルトのTileset
が期待通りに作成されることを確認します。このテストは、Tileset
構造体のデフォルト実装が正しく機能していることを保証するために重要です。- 443-462:
test_bounding_volumes
関数は、異なる形状(ボックス、球、領域)の境界ボリュームの作成をテストします。このテストは、BoundingVolume
構造体が正しく機能していることを保証し、異なるタイプの境界ボリュームが期待通りに作成されることを確認します。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.