AreaMeshFactory ::combineメソッドでもgroup単位で分割するように変更 #285
Conversation
This file contains hidden or 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
Walkthroughシティオブジェクトの即時メッシュ生成を廃し、各オブジェクトの gmlPath を保持したまま (group_id, grid_id=0) でグループ化してから、グループごとに MeshFactory を作成してメッシュ生成・最適化・統合する二段階処理に変更。 Changes
Sequence Diagram(s)sequenceDiagram
participant Input as シティオブジェクト一覧
participant Mapper as GroupMap作成
participant Grouper as グループ化 (group_id, grid_id=0)
participant PerGroup as グループ毎 MeshFactory
participant Merger as GridMergeResult
Note over Input,Mapper: 各 CityObject と gmlPath をペアで収集
Input->>Mapper: CityObject + gmlPath
Mapper->>Grouper: GroupGridIDToObjectsWithPathMap
Note over Grouper: 各オブジェクトの max_lod を計算\nhighest_lod_only フィルタ適用
Grouper->>PerGroup: 各グループのオブジェクト集合
loop グループ毎処理
PerGroup->>PerGroup: MeshFactory を生成
PerGroup->>PerGroup: プライマリ + 子(LOD≥2) を追加 (gmlPath保持)
PerGroup->>PerGroup: メッシュ生成 → 最適化
PerGroup->>Merger: 最適化済みグループメッシュを追加
end
Merger->>Merger: グループ結果をマージして GridMergeResult を返す
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/polygon_mesh/area_mesh_factory.cpp(1 hunks)
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/area_mesh_factory.cpp:195-210
Timestamp: 2025-08-18T07:10:22.941Z
Learning: `area_mesh_factory.cpp`の`gridMerge`と`combine`メソッドでは、`max_lod_in_obj`の計算タイミングが異なり、パフォーマンス特性が微妙に異なる。`gridMerge`では既にグループ分け用に計算済み、`combine`では`highest_lod_only`が有効な場合のみ計算される。
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `AreaMeshFactory::gridMerge`では、`group_id = grid_id * (max_lod_in_specification_ + 1) + max_lod_in_obj`の計算式により、各`group_id`は`grid_id`と`max_lod`の組み合わせから一意に決まる。そのため、`mesh_extractor.cpp`の文脈では`group_id`だけでノード名を構築しても重複は発生しない。
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `area_mesh_factory.cpp`の`gridMerge`では、`group_id`は`grid_id * 100 + max_lod_in_obj`として計算されるため、同一の`group_id`を持つ異なる`grid_id`のペアは存在しない。したがって、`mesh_extractor.cpp`では`group_grid_id.first`のみでノード名を構築しても重複は発生しない。
📚 Learning: 2025-08-18T07:10:22.941Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/area_mesh_factory.cpp:195-210
Timestamp: 2025-08-18T07:10:22.941Z
Learning: `area_mesh_factory.cpp`の`gridMerge`と`combine`メソッドでは、`max_lod_in_obj`の計算タイミングが異なり、パフォーマンス特性が微妙に異なる。`gridMerge`では既にグループ分け用に計算済み、`combine`では`highest_lod_only`が有効な場合のみ計算される。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
📚 Learning: 2025-08-19T07:27:43.111Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `area_mesh_factory.cpp`の`gridMerge`では、`group_id`は`grid_id * 100 + max_lod_in_obj`として計算されるため、同一の`group_id`を持つ異なる`grid_id`のペアは存在しない。したがって、`mesh_extractor.cpp`では`group_grid_id.first`のみでノード名を構築しても重複は発生しない。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
📚 Learning: 2025-08-19T07:27:43.111Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `AreaMeshFactory::gridMerge`では、`group_id = grid_id * (max_lod_in_specification_ + 1) + max_lod_in_obj`の計算式により、各`group_id`は`grid_id`と`max_lod`の組み合わせから一意に決まる。そのため、`mesh_extractor.cpp`の文脈では`group_id`だけでノード名を構築しても重複は発生しない。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
🧬 Code graph analysis (1)
src/polygon_mesh/area_mesh_factory.cpp (2)
src/polygon_mesh/polygon_mesh_utils.cpp (4)
findFirstPolygon(83-97)findFirstPolygon(83-83)getChildCityObjectsRecursive(73-81)getChildCityObjectsRecursive(73-73)src/polygon_mesh/mesh_extractor.cpp (4)
isTypeToSkip(210-217)isTypeToSkip(210-210)shouldContainPrimaryMesh(198-208)shouldContainPrimaryMesh(198-198)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
- GitHub Check: build-and-test (macos-14, arm64)
- GitHub Check: upload-dlls (macos-14, arm64)
- GitHub Check: upload_mobile_dlls (ubuntu-latest, 33, 30.0.3, 23.1.7779620)
- GitHub Check: build-and-test (ubuntu-24.04, x86_64)
- GitHub Check: build-and-test (windows-2022, x86_64)
- GitHub Check: upload-dlls (ubuntu-24.04, x86_64)
- GitHub Check: upload-dlls (macos-14, x86_64)
- GitHub Check: upload-dlls (windows-2022, x86_64)
🔇 Additional comments (1)
src/polygon_mesh/area_mesh_factory.cpp (1)
187-222: グループ化ロジックの実装が正確です。
combineメソッドにgridMergeと同様のグループベースのアプローチが導入されており、PR目的（LOD1テクスチャ適用のためのgroup単位でのメッシュ分割）を達成しています。実装の詳細：
max_lod_in_objの計算（194-202行）により、各オブジェクトが持つ最大LODを判定highest_lod_onlyオプションのフィルタリング（204-210行）が正しく実装されているgroup_id = max_lod_in_objの単純化された計算式（215行）は、grid_idが常に0であるため適切パフォーマンスに関する注意：
以前の実装ではmax_lod_in_objの計算はhighest_lod_onlyが有効な場合のみ実行されていましたが、今回の変更ですべてのオブジェクトに対して実行されるようになりました。これはグループ化に必要な変更ですが、パフォーマンス特性が変わることに留意してください。Based on learnings
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/polygon_mesh/area_mesh_factory.cpp(2 hunks)
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/area_mesh_factory.cpp:195-210
Timestamp: 2025-08-18T07:10:22.941Z
Learning: `area_mesh_factory.cpp`の`gridMerge`と`combine`メソッドでは、`max_lod_in_obj`の計算タイミングが異なり、パフォーマンス特性が微妙に異なる。`gridMerge`では既にグループ分け用に計算済み、`combine`では`highest_lod_only`が有効な場合のみ計算される。
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `AreaMeshFactory::gridMerge`では、`group_id = grid_id * (max_lod_in_specification_ + 1) + max_lod_in_obj`の計算式により、各`group_id`は`grid_id`と`max_lod`の組み合わせから一意に決まる。そのため、`mesh_extractor.cpp`の文脈では`group_id`だけでノード名を構築しても重複は発生しない。
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `area_mesh_factory.cpp`の`gridMerge`では、`group_id`は`grid_id * 100 + max_lod_in_obj`として計算されるため、同一の`group_id`を持つ異なる`grid_id`のペアは存在しない。したがって、`mesh_extractor.cpp`では`group_grid_id.first`のみでノード名を構築しても重複は発生しない。
📚 Learning: 2025-08-18T07:10:22.941Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/area_mesh_factory.cpp:195-210
Timestamp: 2025-08-18T07:10:22.941Z
Learning: `area_mesh_factory.cpp`の`gridMerge`と`combine`メソッドでは、`max_lod_in_obj`の計算タイミングが異なり、パフォーマンス特性が微妙に異なる。`gridMerge`では既にグループ分け用に計算済み、`combine`では`highest_lod_only`が有効な場合のみ計算される。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
📚 Learning: 2025-08-19T07:27:43.111Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `area_mesh_factory.cpp`の`gridMerge`では、`group_id`は`grid_id * 100 + max_lod_in_obj`として計算されるため、同一の`group_id`を持つ異なる`grid_id`のペアは存在しない。したがって、`mesh_extractor.cpp`では`group_grid_id.first`のみでノード名を構築しても重複は発生しない。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
📚 Learning: 2025-08-19T07:27:43.111Z
Learnt from: sevendev
Repo: Synesthesias/libplateau PR: 281
File: src/polygon_mesh/mesh_extractor.cpp:62-64
Timestamp: 2025-08-19T07:27:43.111Z
Learning: `AreaMeshFactory::gridMerge`では、`group_id = grid_id * (max_lod_in_specification_ + 1) + max_lod_in_obj`の計算式により、各`group_id`は`grid_id`と`max_lod`の組み合わせから一意に決まる。そのため、`mesh_extractor.cpp`の文脈では`group_id`だけでノード名を構築しても重複は発生しない。
Applied to files:
src/polygon_mesh/area_mesh_factory.cpp
🧬 Code graph analysis (1)
src/polygon_mesh/area_mesh_factory.cpp (2)
src/polygon_mesh/polygon_mesh_utils.cpp (4)
findFirstPolygon(83-97)findFirstPolygon(83-83)getChildCityObjectsRecursive(73-81)getChildCityObjectsRecursive(73-73)src/polygon_mesh/mesh_extractor.cpp (4)
isTypeToSkip(210-217)isTypeToSkip(210-210)shouldContainPrimaryMesh(198-208)shouldContainPrimaryMesh(198-198)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: build-and-test (ubuntu-24.04, x86_64)
- GitHub Check: build-and-test (windows-2022, x86_64)
- GitHub Check: build-and-test (macos-14, arm64)
- GitHub Check: check-submodule-license
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.
🔗 関連リンク
https://synesthesias.atlassian.net/browse/CPSDK25-134
✅ レビュー前確認項目
✅ マージ前確認項目
🚀 実装内容
LOD1テクスチャを貼る対応でgroup単位でメッシュが分かれていないと撮影がうまくいかないためLib側に修正を入れました
🌐 影響範囲
🛠️ 動作確認
Summary by CodeRabbit
リリースノート