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

3dtiles: MetadataEncoder のユニットテスト #406

Open
ciscorn opened this issue Mar 7, 2024 · 0 comments
Open

3dtiles: MetadataEncoder のユニットテスト #406

ciscorn opened this issue Mar 7, 2024 · 0 comments
Assignees
Labels
Milestone

Comments

@ciscorn
Copy link
Member

ciscorn commented Mar 7, 2024

テストがない

@ciscorn ciscorn changed the title 3dtiles: metadata encoder のユニットテスト 3dtiles: MetadataEncoder のユニットテスト Mar 7, 2024
@ciscorn ciscorn self-assigned this Mar 7, 2024
@nokonoko1203 nokonoko1203 added this to the 納品 milestone Mar 8, 2024
ciscorn added a commit that referenced this issue Mar 8, 2024
## Tasks

- [x] 3D Metadata Spec の Schema と Property tables を組む仕組み
- [x] Code type を ENUM にエンコードする
- [x] STRING (stringOffset) に対応する
- [x] (STRING以外で) array (arrayOffset) に対応する
- [x] STRING + array (arrayOffset) に対応する
- [x] noData 値を適当に設定

## 既知の問題

- テストがない #406
- BOOLEAN (bitstreamでの表現) に対応していない #407
- 複数の地物種類 contents を含めた Tileset
を作ると一部のタイル表示されなくなる(これは今回の変更とは関係なく、以前から発生していた可能性が高い) #408

![stractural
metadata](https://github.com/MIERUNE/PLATEAU-GIS-Converter/assets/5351911/00b89f42-167f-4d25-9be6-830c5b8da74a)

Close #376

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit


- **新機能**
    - Rust Analyzerの保存時チェック機能を無効にする設定を追加しました。
    - 構造メタデータのための`metadata_encoder`を使用して、ダミーメタデータの生成を置き換えました。
    - 特徴属性を特定の形式にエンコードするモジュールを追加しました。
    - メタデータエンコーディングプロセスに新しい機能とデータ構造を追加しました。
    - 可変バイトベクターを指定されたアライメントにパディングする新しい公開関数`add_padding`を追加しました。
- **バグ修正**
-
多角形のリング長を比較するネストされたループを削除し、`ApplyAppearanceTransform`実装内の処理ロジックに影響を与えました。
- **リファクタ**
-
複数のファイルでインポートの再編成、関数への新しいパラメーターの追加、バッファーとアクセサーの命名と扱いの更新、コードの構造と可読性の向上、およびメタデータエンコーディングプロセスのリファクタリングを行いました。
- **スタイル**
    - `Image`構造体の実装を変更して、`BufferView`作成プロセス内で`name`フィールドを"image"に設定しました。
- **機能改善**
-
`DataRequirements`構造体の`key_value`フィールドを`KeyValueSpec::JsonifyObjectsAndArrays`に変更しました。
-
`KeyValueSpec`列挙型を精緻化して、ネストされたオブジェクトと配列の変換に影響を与える`JsonifyObjectsAndArrays`と`JsonifyObjects`を区別しました。

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@ciscorn ciscorn modified the milestones: 2024Q2, Future Mar 15, 2024
@nokonoko1203 nokonoko1203 modified the milestones: Future, 2025 May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants